# HG changeset patch
# User Paul Eggert <eggert@cs.ucla.edu>
# Date 1360339403 28800
# Node ID 9d7698fd5b5f87d0ef989934d3fd542fdc468db7
# Parent  4fd6896488e69b98f992e84f5a66de5a5503e3b4
openpty: fix bug where HAVE_OPENPTY is mistakenly 1

Problem reported by Mats Erik Andersson in
<http://lists.gnu.org/archive/html/bug-gnulib/2013-02/msg00051.html>.
* m4/pty.m4 (gl_FUNC_OPENPTY): Define HAVE_OPENPTY when the
openpty function exists, not merely when we intend to replace it.
This corrects the 2013-01-31 patch, which mistakenly defined
HAVE_OPENPTY even on hosts that lacked it.

diff -r 4fd6896488e6 -r 9d7698fd5b5f ChangeLog
--- a/ChangeLog	Fri Feb 08 07:26:45 2013 -0800
+++ b/ChangeLog	Fri Feb 08 08:03:23 2013 -0800
@@ -1,3 +1,13 @@
+2013-02-08  Paul Eggert  <eggert@cs.ucla.edu>
+
+	openpty: fix bug where HAVE_OPENPTY is mistakenly 1
+	Problem reported by Mats Erik Andersson in
+	<http://lists.gnu.org/archive/html/bug-gnulib/2013-02/msg00051.html>.
+	* m4/pty.m4 (gl_FUNC_OPENPTY): Define HAVE_OPENPTY when the
+	openpty function exists, not merely when we intend to replace it.
+	This corrects the 2013-01-31 patch, which mistakenly defined
+	HAVE_OPENPTY even on hosts that lacked it.
+
 2013-02-07  Paul Eggert  <eggert@cs.ucla.edu>
 
 	secure_getenv: fix include typo
diff -r 4fd6896488e6 -r 9d7698fd5b5f m4/pty.m4
--- a/m4/pty.m4	Fri Feb 08 07:26:45 2013 -0800
+++ b/m4/pty.m4	Fri Feb 08 08:03:23 2013 -0800
@@ -98,6 +98,8 @@
 #endif
 ]])
   if test $ac_cv_have_decl_openpty = yes; then
+    AC_DEFINE([HAVE_OPENPTY], [1],
+      [Define to 1 if the system has the 'openpty' function.])
     dnl The system has openpty.
     dnl Prefer glibc's const-safe prototype, if available.
     AC_CACHE_CHECK([for const-safe openpty signature],
@@ -122,18 +124,14 @@
         ],
         [gl_cv_func_openpty_const=yes], [gl_cv_func_openpty_const=no])
       ])
-  fi
-  if test $gl_cv_func_openpty_const = yes; then
-    HAVE_OPENPTY=1
+    if test $gl_cv_func_openpty_const != yes; then
+      REPLACE_OPENPTY=1
+    fi
   else
-    dnl We need gnulib's openpty.
+    dnl The system does not have openpty.
+    HAVE_OPENPTY=0
     dnl Prerequisites of lib/openpty.c in this case.
-    AC_CHECK_FUNCS([_getpty posix_openpt], [HAVE_OPENPTY=1], [HAVE_OPENPTY=0])
-    REPLACE_OPENPTY=$HAVE_OPENPTY
-  fi
-  if test $HAVE_OPENPTY = 1; then
-    AC_DEFINE([HAVE_OPENPTY], [1],
-      [Define to 1 if the system has the 'openpty' function.])
+    AC_CHECK_FUNCS([_getpty posix_openpt])
   fi
 ])