pkgsrc-Changes-HG archive

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

[pkgsrc/trunk]: pkgsrc/misc/rlwrap rlwrap: Properly include <util.h> for open...



details:   https://anonhg.NetBSD.org/pkgsrc/rev/43d8a4fa4843
branches:  trunk
changeset: 312901:43d8a4fa4843
user:      leot <leot%pkgsrc.org@localhost>
date:      Tue Sep 18 14:17:26 2018 +0000

description:
rlwrap: Properly include <util.h> for openpty(3)

Previously on NetBSD and probably other system that has openpty(3)
in util.h the compilation lead to a:

 ptytty.c: In function 'ptytty_get_pty':
 ptytty.c:72:7: warning: implicit declaration of function 'openpty' [-Wimplicit-function-declaration]
    if (openpty(&pfd, fd_tty, tty_name, NULL, NULL) != -1) {
        ^~~~~~~

due a missing <util.h> inclusion.

Add a patch for configure.ac (and auto* tools to USE_TOOLS) in
order to address that.

Originally reported by Matthew Mondor via PR pkg/46190 and thanks
to Matthew and <wiz> for their analysis!

diffstat:

 misc/rlwrap/Makefile                   |   6 +++++-
 misc/rlwrap/distinfo                   |   3 ++-
 misc/rlwrap/patches/patch-configure.ac |  19 +++++++++++++++++++
 3 files changed, 26 insertions(+), 2 deletions(-)

diffs (62 lines):

diff -r d07c54eda983 -r 43d8a4fa4843 misc/rlwrap/Makefile
--- a/misc/rlwrap/Makefile      Tue Sep 18 09:49:38 2018 +0000
+++ b/misc/rlwrap/Makefile      Tue Sep 18 14:17:26 2018 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.19 2018/08/22 09:45:46 wiz Exp $
+# $NetBSD: Makefile,v 1.20 2018/09/18 14:17:26 leot Exp $
 #
 
 DISTNAME=      rlwrap-0.43
@@ -14,6 +14,7 @@
 
 GNU_CONFIGURE=         yes
 USE_TOOLS+=            gmake perl:run
+USE_TOOLS+=            autoreconf autoconf automake
 
 REPLACE_PERL+= filters/censor_passwords
 REPLACE_PERL+= filters/count_in_prompt
@@ -37,6 +38,9 @@
 
 TEST_TARGET=           check
 
+pre-configure:
+       cd ${WRKSRC} && autoreconf -fi
+
 .include "options.mk"
 
 BUILDLINK_API_DEPENDS.readline+=readline>=4.2
diff -r d07c54eda983 -r 43d8a4fa4843 misc/rlwrap/distinfo
--- a/misc/rlwrap/distinfo      Tue Sep 18 09:49:38 2018 +0000
+++ b/misc/rlwrap/distinfo      Tue Sep 18 14:17:26 2018 +0000
@@ -1,6 +1,7 @@
-$NetBSD: distinfo,v 1.6 2017/04/10 20:30:15 leot Exp $
+$NetBSD: distinfo,v 1.7 2018/09/18 14:17:26 leot Exp $
 
 SHA1 (rlwrap-0.43.tar.gz) = 0c0a3a6012a003d4ae3a44e9f82f9090b4ac4650
 RMD160 (rlwrap-0.43.tar.gz) = 8c122eb2acb8963132be744aae0f6d416169877e
 SHA512 (rlwrap-0.43.tar.gz) = 4a1440b2eef3a1664c6381eb09c19f4030de2c0ef4f1f3509f7d6e130d3c5e9c3ceb00fc6ea6052b22bbe1f6b162f72e0b47564df34ccecc5220f195c22e86a1
 Size (rlwrap-0.43.tar.gz) = 313139 bytes
+SHA1 (patch-configure.ac) = b5550f832f276388efa1544ac4f860c338747e2c
diff -r d07c54eda983 -r 43d8a4fa4843 misc/rlwrap/patches/patch-configure.ac
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/misc/rlwrap/patches/patch-configure.ac    Tue Sep 18 14:17:26 2018 +0000
@@ -0,0 +1,19 @@
+$NetBSD: patch-configure.ac,v 1.1 2018/09/18 14:17:26 leot Exp $
+
+Really check for <util.h> needed by openpty(3) at least on NetBSD.
+
+Please note that src/rlwrap.h already checked for HAVE_UTIL_H in
+src/rlwrap.h but actually HAVE_UTIL_H was never defined because
+util.h was not added to the AC_CHECK_HEADERS().
+
+--- configure.ac.orig  2017-03-03 09:41:58.000000000 +0000
++++ configure.ac
+@@ -102,7 +102,7 @@ AC_CHECK_PROG(STRIP,strip,strip,true)
+ # Checks for header files.
+ AC_HEADER_STDC
+ AC_HEADER_SYS_WAIT
+-AC_CHECK_HEADERS([errno.h fcntl.h libgen.h libutil.h stdlib.h string.h sched.h sys/ioctl.h sys/wait.h sys/resource.h stddef.h ])
++AC_CHECK_HEADERS([errno.h fcntl.h libgen.h libutil.h util.h stdlib.h string.h sched.h sys/ioctl.h sys/wait.h sys/resource.h stddef.h ])
+ AC_CHECK_HEADERS([termios.h unistd.h stdint.h time.h getopt.h regex.h curses.h termcap.h ])
+ 
+ AC_CHECK_HEADERS([ term.h  ncurses/term.h], , ,



Home | Main Index | Thread Index | Old Index