World: r4wp

>> t: task [] [wait 2 print "Done"]
>> do t print "Go"
Begin Task
Go
>> Done
End Task

Tasks sort of work. They actually do start, but there are a lot of 
unresolved problems in the internals, and some of what is supposed 
to make them work properly hasn't been done yet, and some stuff like 
the synchronization and sharing models hasn't even been designed 
yet. So it's not really recommended that you use them yet.

Is this the expected / required result of a JOIN on two TYPESET!s?

    >> join to-typeset [pair!] to-typeset [integer!]
    == "pair!integer!"

I think so. Word! values turn into strings as well. You can use UNION 
to join typesets though.

Thanks....It seems odd though, given the different (and for me unexpected) 
results here:
    join  to-typeset [integer!] []
    join  [] to-typeset [integer!]

Yeah, sorry, the fake typesets in R2 are actually blocks. The real 
typesets in R3 aren't series at all.

I faked them well enough in R2 to use them in most cases in an R3 
style, but not well enough to make them fail to do the stuff they 
are supposed to fail to do in R3. Same with closures.

As long as your are happy.....Without looking at the code, I assume 
typesets in R3 are specialised bitsets.

Still, would be nice for the behavior to be a little more consistent.

They are specialized bitsets, so specialized that they fit within 
a value slot rather than being an external structure. But JOIN forms 
non-series, so they are consistent with bitsets in that JOIN does 
the same thing with them.

However, you can't use bitsets to fake them in R2.

Hi all. Is this an appropraite room for discussing an R3 patch?

appropriate*

Hi Marc! Yes, it certainly is.

Neat. Before I get to that: is task launching working on POSIX systems?

No.

Okay, good to know.

Right, here it is: https://github.com/0branch/r3/commit/5493e97c1afaa3848bf448cb49ccf03a1632302d

Hopefully the commit message is gives a clear explanation (and explains 
the motivation).

s/ is //

Brief restatement: BROWSE isn't working on OSX -- the current implementation 
tries two system() calls: xdg-open and x-www-browser

This patch uses /usr/bin/open on OSX, maintains the previous handlers 
on other platforms. In addition, I replaced the system() calls with 
fork+exec to avoid firing up another shell; one byproduct of this 
is that exec errors no longer print to the console (though this could 
have been hackishly solved by adding some redirects to the system() 
call string in the existing codebase).

Did you test this on Linux as well?

Not yet, can do so now.

Please do.

Firing up the VM, give me 5 mins.

No hurries :)

Look here http://issue.cc/r3/1921before you do this. Make sure the 
security constraints mentioned there are taken into account.

(Works on my Fedora instance.)

Thanks for testing that.

Basically: looks fine, thanks a lot for working on this.

BrianH: I don't think this change is an ideal solution, but it seems 
to be an improvement on prior behaviour.

If you can find a way to be more specific on OSX, so that _only_ 
URLs get handled, that would be great. OTOH, xdg-open is already 
rather versatile as well (only I fear that OSX's open is even more 
featureful).

One problem with the existing implementation is that xdg-open and 
x-www-browser are searched for in the PATH, so executables with the 
same names in a PATH directory that takes precedence will yield different 
behaviour.

Andreas: yeah, file:/// will open in Finder with this pathc.

patch*.

(So it's not constrained to the browser.)

That's the same with xdg-open.

Right.

Yeah. It looks like it just calls open, which can be a bit of a hole. 
It's better for now but we need to come up with a better solution 
in the long run.

Agreed.

Hmm, why does BROWSE accept NONE?

BrianH: I don't mean to present this patch as something it's not. 
To clarify: still a hacky solution, but at least it brings the current 
behaviour to OS X.

No, agreed. It's still nowhere near as bad as the #1921 approach, 
or the last ticket that requested it. But we need to make sure to 
come up with something better in the long run.

For sure.

I might patch this NONE case unless anyone has a good reason to leave 
it in place?

(At the moment, you get usage info on stderr. The current r3 just 
segfaults on Linux.)

It's none propagation I guess. Because dealing with or ignoring nones 
is easier to do in Rebol than writing conditional code around everything 
to avoid triggering errors.

Okay, so I guess we just need to handle the NONE rather than attempting 
to browse it

The BROWSE code should explicitly do nothing if passed a NONE. It 
certainly shouldn't segfault.

If it doesn't explicitly do nothing, that's a bug.