[-,alsa-lib,1/1] include/global.h: missing struct on FreeBSD
diff mbox

Message ID 54b0f97b.ca51c20a.1b71.ffffc3db@mx.google.com
State New, archived
Headers show

Commit Message

rumko@rumko.net Jan. 10, 2015, 10:04 a.m. UTC
From: Rumko <rumko@rumko.net>

On FreeBSD, struct timespec is not accessible through <time.h>,
but through <sys/tiespec.h>. Add specific include for that header
on that platform only.

Signed-off-by: Rumko <rumko@rumko.net>

Comments

rumko@rumko.net Jan. 11, 2015, 11:17 a.m. UTC | #1
On Sunday 11 January 2015 11:27:52 Takashi Iwai wrote:
> At Sat, 10 Jan 2015 11:04:59 +0100,
> 
> rumko@rumko.net wrote:
> > From: Rumko <rumko@rumko.net>
> > 
> > On FreeBSD, struct timespec is not accessible through <time.h>,
> > but through <sys/tiespec.h>. Add specific include for that header
> > on that platform only.
> > 
> > Signed-off-by: Rumko <rumko@rumko.net>
> 
> If you give your sign-off, please give the full real name.

Sorry about that, was just following
http://www.alsa-project.org/main/index.php/GIT_Server#Occasional_Developers
It specifically adds the -s flag to format-patch as instructions for generating 
the patchset.

Should I resubmit without it?

> thanks,
> 
> Takashi
Takashi Iwai Jan. 11, 2015, 3:57 p.m. UTC | #2
At Sun, 11 Jan 2015 12:17:31 +0100,
Rumko wrote:
> 
> On Sunday 11 January 2015 11:27:52 Takashi Iwai wrote:
> > At Sat, 10 Jan 2015 11:04:59 +0100,
> > 
> > rumko@rumko.net wrote:
> > > From: Rumko <rumko@rumko.net>
> > > 
> > > On FreeBSD, struct timespec is not accessible through <time.h>,
> > > but through <sys/tiespec.h>. Add specific include for that header
> > > on that platform only.
> > > 
> > > Signed-off-by: Rumko <rumko@rumko.net>
> > 
> > If you give your sign-off, please give the full real name.
> 
> Sorry about that, was just following
> http://www.alsa-project.org/main/index.php/GIT_Server#Occasional_Developers
> It specifically adds the -s flag to format-patch as instructions for generating 
> the patchset.
> 
> Should I resubmit without it?

We follow the policy of Linux kernel development for the user-space
stuff in principle.  See Documentation/SubmittingPatches of Linux
driver tree, "Sign your work" section.  You need to give the real
name.


thanks,

Takashi
rumko@rumko.net Jan. 11, 2015, 4:52 p.m. UTC | #3
On Sunday 11 January 2015 16:57:39 Takashi Iwai wrote:
<snip>
> We follow the policy of Linux kernel development for the user-space
> stuff in principle.  See Documentation/SubmittingPatches of Linux
> driver tree, "Sign your work" section.  You need to give the real
> name.

Ah, in that case, sorry to have bothered you. As far as I'm concerned, this 
thread can be closed.

Thank you

> thanks,
> 
> Takashi

Patch
diff mbox

diff --git a/include/global.h b/include/global.h
index 16a26dc..70be1f9 100644
--- a/include/global.h
+++ b/include/global.h
@@ -31,6 +31,10 @@ 
 /* for timeval and timespec */
 #include <time.h>
 
+#if defined(__FreeBSD__)
+#include <sys/timespec.h>
+#endif
+
 #ifdef __cplusplus
 extern "C" {
 #endif