On Wed, 2014-10-08 at 10:20 +0200, Thomas Schwinge wrote: > Hi! > > On Tue, 07 Oct 2014 18:42:06 +0200, Svante Signell <svante.sign...@gmail.com> > wrote: > > libgroove fails to build on GNU/Hurd due to a name clash with OSX, both > > are defining the __MACH__ keyword. > > > --- a/grooveplayer/osx_time_shim.h 2014-09-25 17:26:09.000000000 +0200 > > +++ b/grooveplayer/osx_time_shim.h 2014-10-07 18:27:30.000000000 +0200 > > @@ -8,7 +8,7 @@ > > > > #ifndef GROOVE_MACH_TIME_H_INCLUDED > > #define GROOVE_MACH_TIME_H_INCLUDED > > -#ifdef __MACH__ > > +#if defined(__MACH__) && !defined(__GNU__) > > Instead of masking out the false positive, how about explicitly stating: > »#if defined(__MACH__) && defined(__APPLE__)«. That's how it is usually > written, if I remember correctly. > > That said, just to confirm: it is expected that this file, > osx_time_shim.h, is included even for non-OSX builds?
Thanks Thomas I'll remember this for other packages. Yes, it is included for non-OSX builds too. _______________________________________________ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers