<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 7, 2014 at 10:31 AM, Pino Toscano <span dir="ltr"><<a href="mailto:pino@debian.org" target="_blank">pino@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2014-10-07 18:42, Svante Signell wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Source: libgroove<br>
Version: 4.2.0-1<br>
Severity: important<br>
Tags: patch<br>
User: <a href="mailto:debian-hurd@lists.debian.org" target="_blank">debian-hurd@lists.debian.org</a><br>
Usertags: hurd<br>
<br>
Hi,<br>
<br>
libgroove fails to build on GNU/Hurd  due to a name clash with OSX, both<br>
are defining the __MACH__ keyword.<br>
</blockquote>
<br></span>
--- a/grooveplayer/osx_time_shim.h      2014-09-25 17:26:09.000000000 +0200<br>
+++ b/grooveplayer/osx_time_shim.h      2014-10-07 18:27:30.000000000 +0200<br>
@@ -8,7 +8,7 @@<br>
<br>
 #ifndef GROOVE_MACH_TIME_H_INCLUDED<br>
 #define GROOVE_MACH_TIME_H_INCLUDED<br>
-#ifdef __MACH__<br>
+#if defined(__MACH__) && !defined(__GNU__)<br>
<br>
 #include <sys/types.h><br>
 #include <sys/_types/_timespec.h><br>
<br>
This file is definitely OSX-specific, so the above preprocessor<br>
condition is too loose; it should be __MACH__ and __APPLE__, or simply<br>
just __APPLE__.</blockquote><div><br></div><div>Thank you for the report and the patch. libgroove upstream released 4.2.1 which resolves this problem. The libgroove Debian package is updated to this new release and is ready for upload.</div></div></div></div>