bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] tcgetsid: fix cygwin header bug


From: Eric Blake
Subject: [PATCH] tcgetsid: fix cygwin header bug
Date: Wed, 29 Feb 2012 06:36:16 -0700

Cygwin added tcgetsid in 1.7.10, but with int instead of pid_t:
http://cygwin.com/ml/cygwin-patches/2012-q1/msg00031.html
For now, since cygwin's pid_t is int, I'm not worrying about
correcting the return type.

* lib/termios.in.h (includes) [Cygwin]: Ensure pid_t is defined.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog        |    3 +++
 lib/termios.in.h |    6 ++++++
 2 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index cc12b74..8f48d9a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2012-02-29  Eric Blake  <address@hidden>

+       tcgetsid: fix cygwin header bug
+       * lib/termios.in.h (includes) [Cygwin]: Ensure pid_t is defined.
+
        docs: update cygwin progress
        * doc/posix-functions/llround.texi (llround): Added in cygwin
        1.7.8.
diff --git a/lib/termios.in.h b/lib/termios.in.h
index 85d1c60..3836e2d 100644
--- a/lib/termios.in.h
+++ b/lib/termios.in.h
@@ -32,6 +32,12 @@ extern "C" {
 }
 #endif

+/* On Cygwin 1.7.11, tcgetsid returns int instead of pid_t; at least
+   they are the same size on that platform.  */
+#ifdef __CYGWIN__
+# include <sys/types.h>
+#endif
+
 /* The include_next requires a split double-inclusion guard.  */
 #if @HAVE_TERMIOS_H@
 # @INCLUDE_NEXT@ @NEXT_TERMIOS_H@
-- 
1.7.7.6




reply via email to

[Prev in Thread] Current Thread [Next in Thread]