World: r4wp

browse none doesn't work right on Win32 unless IE is your default 
browser. It behaves differently wrong for every other default browser.

Aha, interesting.

Add that to the browse none ticket, Andreas, we can say "All Platforms".

So we probably better rather remove this feature altogether.

> MarcS: one open question re your original pull is if we should 
use the __APPLE__ define

> (is that defined for iOS as well?) or rather use the R3-specific 
platform defines.

It would be worth taking none out of the argument spec and marking 
the ticket as a problem until we figure out how to do this correctly.

Stylistically I'd imagine Rebol's defines are preferred?

Yes please, if we have an appropriate one. And if we don't, we should.

But are they reliable? I can't see reference to either __APPLE__ 
or __MACH__ in the codebase

They are build-driven.

You explicitly build for OSX, then the platform defines are set.

TO_MAC, right?

That isn't set correctly

TO_OSXI and TO_OSX

Oh, thanks.

sys-system.h is unused, and only there to cause confusion :)

:)

The multitude of defines for platforms in the same "major" group 
is a bit of an annoyance as well.

hmm, so we'd need a disjunction

Yes, exactly.

TO_OSX || TO_OSXI

Is TO_OSX for all OSX or just PPC?

that's what this comment indicates

src/include/reb-config.h:#ifdef TO_OSXI					// OSX/Intel
src/include/reb-config.h:#ifdef TO_OSX					// OSX/PPC

TO_OSX is osx-ppc, TO_OSXI is osx-intel

OK. Good to know.

And we have no common defines for "all of ..." at the moment, unfortunately.

My only Mac is a G5.

Hmm

-#ifdef __APPLE__
+#if defined(TO_OSX) || defined(TO_OSXI)

neither are defined (unless I screwed that up!)

The only references I see in the codebase are the above ifdefs (reb-config.h)

Should be defined and passed along in your makefile.

$ grep -R TO_OSX *
src/include/reb-config.h:#ifdef TO_OSXI					// OSX/Intel
src/include/reb-config.h:#ifdef TO_OSX					// OSX/PPC

in a make/makefile for osx-intel:
TO_OS?= TO_OSXI
...
RFLAGS= -c -D$(TO_OS) -DREB_API  $(RAPI_FLAGS) $I
HFLAGS= -c -D$(TO_OS) -DREB_CORE $(HOST_FLAGS) $I

hmm

oh, mine reads TO_LINUX

shouldn't make prep have set that correctly?

make make OS_ID=0.2.5

Is there a reason that this isn't autodetected?

No good reason. But that's just how the build currently works.

Okay, thanks for the tip.

To support building Rebol for old crappy platforms with stupid make.

At least that's the stated reason. There is likely a better approach.

https://github.com/0branch/r3/commits/browse-macosx

http://issue.cc/r3/1991- BROWSE none

https://github.com/0branch/r3/commit/8eea3cf8217bd8f05ea5190fa81f5e8d8ed41cad

MarcS, with mainline R3 browse doesn't currently work on OSX, right?

Right

Unless, I guess, you have XQuartz installed with xdg-open or x-www-browse