On 31.12.2012 03:33 (UTC+2), Steven Chamberlain wrote: > Hi Dirk, > > On 31/12/12 02:06, Dirk Eddelbuettel wrote: >> | > #elif defined(linux) || defined(__linux) || defined(__FreeBSD__) > > Sometimes you are really testing for the type of libc on the system, not > the kernel. For that, __GLIBC__ is supposed to be defined on all > GNU/k*BSD variants. There is a bit more info at: > > http://glibc-bsd.alioth.debian.org/porting/PORTING > >> | > >> | > in two place. If memory serves, there were a) similar handles for >> NetBSD and >> | > OpenBSD I should add. And which one do you guys need for the BSD >> kernel on >> | > Debian? >> | >> | Jep I'm seeing __NetBSD__ and __OpenBSD__ in lots of source. > > Plenty of examples of that here: > > http://codesearch.debian.net/search?q=defined.*BSD.*BSD.*BSD > or > http://codesearch.debian.net/search?q=defined.*BSD__ > > Generally you have to list all of them :( > > > But it is probably more portable in the long term to test for specific > features that you want to use, wherever it makes sense. > >> | uses __FreeBSD_kernel__ which -- if memory serves me correctly -- be >> | also defined in newer FreeBSDs. >> >> ... you are telling me I also need this? > > If you are really testing for kernels, I think it is preferred that you > test for __FreeBSD_kernel__, instead of just __FreeBSD__ which is a bit > ambigious, and only defined on a pure FreeBSD (with BSD libc) system.
I just tried to build Rcpp with __FreeBSD_kernel__ on a recent FreeBSD 10.0-CURRENT (newest developer version) and it does not resolve. On 'pure' FreeBSD there is only __FreeBSD__ defined, not a kernel specific version. I think there are much more 'pure' FreeBSD systems than linux ones with FreeBSD kernel, so it would be nice, if you would pick up at least the __FreeBSD__ symbol. Thanks and greetings, Rainer > Regards, > -- To UNSUBSCRIBE, email to debian-bsd-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/50e13516.3060...@gwdg.de