• Home
  • Script library
  • AltME Archive
  • Mailing list
  • Articles Index
  • Site search
 

World: r4wp

[!REBOL3] General discussion about REBOL 3

MarcS
10-Mar-2013
[1571x3]
But xdg-open takes me to http:///
(Invalid adress.)
Brian: but pull requests should still go to the main repo?
Andreas
10-Mar-2013
[1574]
Yes.
MarcS
10-Mar-2013
[1575x2]
address*
(Sorry for the 20 questions.)
Andreas
10-Mar-2013
[1577]
Glad if we can help :)
BrianH
10-Mar-2013
[1578]
browse none doesn't work right on Win32 unless IE is your default 
browser. It behaves differently wrong for every other default browser.
MarcS
10-Mar-2013
[1579]
Aha, interesting.
BrianH
10-Mar-2013
[1580]
Add that to the browse none ticket, Andreas, we can say "All Platforms".
Andreas
10-Mar-2013
[1581x2]
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.
BrianH
10-Mar-2013
[1583]
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.
MarcS
10-Mar-2013
[1584]
Stylistically I'd imagine Rebol's defines are preferred?
BrianH
10-Mar-2013
[1585]
Yes please, if we have an appropriate one. And if we don't, we should.
MarcS
10-Mar-2013
[1586]
But are they reliable? I can't see reference to either __APPLE__ 
or __MACH__ in the codebase
Andreas
10-Mar-2013
[1587x2]
They are build-driven.
You explicitly build for OSX, then the platform defines are set.
MarcS
10-Mar-2013
[1589x2]
TO_MAC, right?
That isn't set correctly
Andreas
10-Mar-2013
[1591]
TO_OSXI and TO_OSX
MarcS
10-Mar-2013
[1592]
Oh, thanks.
Andreas
10-Mar-2013
[1593]
sys-system.h is unused, and only there to cause confusion :)
MarcS
10-Mar-2013
[1594]
:)
Andreas
10-Mar-2013
[1595]
The multitude of defines for platforms in the same "major" group 
is a bit of an annoyance as well.
MarcS
10-Mar-2013
[1596]
hmm, so we'd need a disjunction
Andreas
10-Mar-2013
[1597]
Yes, exactly.
MarcS
10-Mar-2013
[1598]
TO_OSX || TO_OSXI
BrianH
10-Mar-2013
[1599]
Is TO_OSX for all OSX or just PPC?
MarcS
10-Mar-2013
[1600x2]
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
Andreas
10-Mar-2013
[1602]
TO_OSX is osx-ppc, TO_OSXI is osx-intel
BrianH
10-Mar-2013
[1603]
OK. Good to know.
Andreas
10-Mar-2013
[1604]
And we have no common defines for "all of ..." at the moment, unfortunately.
BrianH
10-Mar-2013
[1605]
My only Mac is a G5.
MarcS
10-Mar-2013
[1606x4]
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)
Andreas
10-Mar-2013
[1610]
Should be defined and passed along in your makefile.
MarcS
10-Mar-2013
[1611]
$ grep -R TO_OSX *
src/include/reb-config.h:#ifdef TO_OSXI					// OSX/Intel
src/include/reb-config.h:#ifdef TO_OSX					// OSX/PPC
Andreas
10-Mar-2013
[1612]
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
MarcS
10-Mar-2013
[1613x3]
hmm
oh, mine reads TO_LINUX
shouldn't make prep have set that correctly?
Andreas
10-Mar-2013
[1616]
make make OS_ID=0.2.5
MarcS
10-Mar-2013
[1617]
Is there a reason that this isn't autodetected?
Andreas
10-Mar-2013
[1618]
No good reason. But that's just how the build currently works.
MarcS
10-Mar-2013
[1619]
Okay, thanks for the tip.
BrianH
10-Mar-2013
[1620]
To support building Rebol for old crappy platforms with stupid make.