# HG changeset patch
# User Pavel Fedin <p.fedin@samsung.com>
# Date 1435735091 -10800
# Node ID 674bd77044974c3faaa5fec50073cded7ca956e4
# Parent  dce9e46372946d7a9ab606496b11eaab7962e4c0
fix pty related tests issues on Windows

* lib/grantpt.c (grantpt): grantpt.c seems to be never used on Windows,
however it's still present in tests/ subdirectory of the final project.
Therefore avoid it to pass `make check`.
* tests/test-openpty.c (main): Windows has no PTYs and gnulib's openpty()
will just return -ENOSYS, so avoid this non applicable test allowing
the build to proceed.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2015-07-01  Pavel Fedin  <p.fedin@samsung.com>
+
+	fix pty related tests issues on Windows (trivial)
+	* lib/grantpt.c (grantpt): grantpt.c seems to be never used on Windows,
+	however it's still present in tests/ subdirectory of the final project.
+	Therefore avoid it to pass `make check`.
+	* tests/test-openpty.c (main): Windows has no PTYs and gnulib's openpty()
+	will just return -ENOSYS, so avoid this non applicable test allowing
+	the build to proceed.
+
 2015-07-01  Pádraig Brady  <P@draigBrady.com>
 
 	acl: fix definition of acl_from_mode on FreeBSD
diff --git a/lib/grantpt.c b/lib/grantpt.c
--- a/lib/grantpt.c
+++ b/lib/grantpt.c
@@ -46,9 +46,10 @@
 int
 grantpt (int fd)
 {
-#if defined __OpenBSD__
+#if defined __OpenBSD__ || defined _WIN32
   /* On OpenBSD, master and slave of a pseudo-terminal are allocated together,
-     through an ioctl on /dev/ptm.  There is no need for grantpt().  */
+     through an ioctl on /dev/ptm.  On Windows there are no ptys.
+     Therefore in either case there is no need for grantpt().  */
   return 0;
 #else
   /* This function is most often called from a process without 'root'
diff --git a/tests/test-openpty.c b/tests/test-openpty.c
--- a/tests/test-openpty.c
+++ b/tests/test-openpty.c
@@ -34,6 +34,7 @@
 main ()
 {
   {
+#ifndef _WIN32
     int master;
     int slave;
 
@@ -103,6 +104,7 @@
     /* Close the master side before the slave side gets closed.
        This is necessary on Mac OS X 10.4.11.  */
     close (master);
+#endif
   }
 
   return 0;