World: r4wp

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.

Just return unset, I guess.

Okay, fixing now.

not R_NONE?

No, that returns none. Whatever BROWSE normally returns.

It normally returns NONE.

R_NONE

But I think the original intent was for NONE to open an empty browser. 
Can't find any references for that.