Mailing List Archive

https://github.com/python/cpython/commit/f201d261cf53365b5769a434ca2bb21a892bd23f
commit: f201d261cf53365b5769a434ca2bb21a892bd23f
branch: main
author: Christian Heimes <christian@python.org>
committer: tiran <christian@python.org>
date: 2021-11-21T10:45:31+01:00
summary:

bpo-45847: Port grp, spwd, termios, resource, syslog to PY_STDLIB_MOD (GH-29668)

files:
M Modules/Setup.stdlib.in
M configure
M configure.ac
M pyconfig.h.in
M setup.py

diff --git a/Modules/Setup.stdlib.in b/Modules/Setup.stdlib.in
index fa553fb524f56..495cfb47ac231 100644
--- a/Modules/Setup.stdlib.in
+++ b/Modules/Setup.stdlib.in
@@ -61,8 +61,14 @@
# Modules with some UNIX dependencies
#

-# Linux and FreeBSD, needs sys/soundcard.h or linux/soundcard.h
+@MODULE_GRP_TRUE@grp grpmodule.c
+# needs sys/soundcard.h or linux/soundcard.h (Linux, FreeBSD)
@MODULE_OSSAUDIODEV_TRUE@ossaudiodev ossaudiodev.c
+@MODULE_RESOURCE_TRUE@resource resource.c
+# AIX has shadow passwords, but does not provide getspent API
+@MODULE_SPWD_TRUE@spwd spwdmodule.c
+@MODULE_SYSLOG_TRUE@syslog syslogmodule.c
+@MODULE_TERMIOS_TRUE@termios termios.c


############################################################################
diff --git a/configure b/configure
index 7d2ef8f912073..45c6eb4b845ab 100755
--- a/configure
+++ b/configure
@@ -658,10 +658,20 @@ MODULE__ELEMENTTREE_FALSE
MODULE__ELEMENTTREE_TRUE
MODULE_PYEXPAT_FALSE
MODULE_PYEXPAT_TRUE
+MODULE_TERMIOS_FALSE
+MODULE_TERMIOS_TRUE
+MODULE_SYSLOG_FALSE
+MODULE_SYSLOG_TRUE
+MODULE_SPWD_FALSE
+MODULE_SPWD_TRUE
MODULE__SCPROXY_FALSE
MODULE__SCPROXY_TRUE
+MODULE_RESOURCE_FALSE
+MODULE_RESOURCE_TRUE
MODULE_OSSAUDIODEV_FALSE
MODULE_OSSAUDIODEV_TRUE
+MODULE_GRP_FALSE
+MODULE_GRP_TRUE
MODULE__DATETIME_FALSE
MODULE__DATETIME_TRUE
MODULE_TIME_FALSE
@@ -8196,7 +8206,7 @@ sys/times.h sys/types.h sys/uio.h sys/un.h sys/utsname.h sys/wait.h pty.h \
libutil.h sys/resource.h netpacket/packet.h sysexits.h bluetooth.h \
linux/tipc.h linux/random.h spawn.h util.h alloca.h endian.h \
sys/endian.h sys/sysmacros.h linux/memfd.h linux/wait.h sys/memfd.h \
-sys/mman.h sys/eventfd.h linux/soundcard.h sys/soundcard.h
+sys/mman.h sys/eventfd.h linux/soundcard.h sys/soundcard.h syslog.h
do :
as_ac_Header=`$as_echo "ac_cv_header_$ac_header" | $as_tr_sh`
ac_fn_c_check_header_mongrel "$LINENO" "$ac_header" "$as_ac_Header" "$ac_includes_default"
@@ -12658,7 +12668,7 @@ for ac_func in alarm accept4 setitimer getitimer bind_textdomain_codeset chown \
explicit_memset faccessat fchmod fchmodat fchown fchownat \
fdwalk fexecve fdopendir fork fpathconf fstatat ftime ftruncate futimesat \
futimens futimes gai_strerror getentropy \
- getgrgid_r getgrnam_r \
+ getgrgid getgrgid_r getgrnam_r \
getgrouplist getgroups getlogin getloadavg getpeername getpgid getpid \
getpriority getresuid getresgid getpwent getpwnam_r getpwuid_r getspnam getspent getsid getwd \
if_nameindex \
@@ -19551,6 +19561,45 @@ fi



+ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module grp" >&5
+$as_echo_n "checking for stdlib extension module grp... " >&6; }
+ case $py_stdlib_not_available in #(
+ *grp*) :
+ py_cv_module_grp=n/a ;; #(
+ *) :
+
+ if true; then :
+ if test "$ac_cv_func_getgrgid" = yes -o "$ac_cv_func_getgrgid_r" = yes; then :
+ py_cv_module_grp=yes
+else
+ py_cv_module_grp=missing
+fi
+else
+ py_cv_module_grp=disabled
+
+fi
+
+ ;;
+esac
+ as_fn_append MODULE_BLOCK "MODULE_GRP=$py_cv_module_grp$as_nl"
+ if test "x$py_cv_module_grp" = xyes; then :
+
+
+
+
+fi
+ if test "$py_cv_module_grp" = yes; then
+ MODULE_GRP_TRUE=
+ MODULE_GRP_FALSE='#'
+else
+ MODULE_GRP_TRUE='#'
+ MODULE_GRP_FALSE=
+fi
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: result: $py_cv_module_grp" >&5
+$as_echo "$py_cv_module_grp" >&6; }
+
+
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module ossaudiodev" >&5
$as_echo_n "checking for stdlib extension module ossaudiodev... " >&6; }
case $py_stdlib_not_available in #(
@@ -19590,6 +19639,45 @@ fi
$as_echo "$py_cv_module_ossaudiodev" >&6; }


+ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module resource" >&5
+$as_echo_n "checking for stdlib extension module resource... " >&6; }
+ case $py_stdlib_not_available in #(
+ *resource*) :
+ py_cv_module_resource=n/a ;; #(
+ *) :
+
+ if true; then :
+ if test "$ac_cv_header_sys_resource_h" = yes; then :
+ py_cv_module_resource=yes
+else
+ py_cv_module_resource=missing
+fi
+else
+ py_cv_module_resource=disabled
+
+fi
+
+ ;;
+esac
+ as_fn_append MODULE_BLOCK "MODULE_RESOURCE=$py_cv_module_resource$as_nl"
+ if test "x$py_cv_module_resource" = xyes; then :
+
+
+
+
+fi
+ if test "$py_cv_module_resource" = yes; then
+ MODULE_RESOURCE_TRUE=
+ MODULE_RESOURCE_FALSE='#'
+else
+ MODULE_RESOURCE_TRUE='#'
+ MODULE_RESOURCE_FALSE=
+fi
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: result: $py_cv_module_resource" >&5
+$as_echo "$py_cv_module_resource" >&6; }
+
+
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module _scproxy" >&5
$as_echo_n "checking for stdlib extension module _scproxy... " >&6; }
case $py_stdlib_not_available in #(
@@ -19629,6 +19717,123 @@ fi
$as_echo "$py_cv_module__scproxy" >&6; }


+ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module spwd" >&5
+$as_echo_n "checking for stdlib extension module spwd... " >&6; }
+ case $py_stdlib_not_available in #(
+ *spwd*) :
+ py_cv_module_spwd=n/a ;; #(
+ *) :
+
+ if true; then :
+ if test "$ac_cv_func_getspent" = yes -o "$ac_cv_func_getspnam" = yes; then :
+ py_cv_module_spwd=yes
+else
+ py_cv_module_spwd=missing
+fi
+else
+ py_cv_module_spwd=disabled
+
+fi
+
+ ;;
+esac
+ as_fn_append MODULE_BLOCK "MODULE_SPWD=$py_cv_module_spwd$as_nl"
+ if test "x$py_cv_module_spwd" = xyes; then :
+
+
+
+
+fi
+ if test "$py_cv_module_spwd" = yes; then
+ MODULE_SPWD_TRUE=
+ MODULE_SPWD_FALSE='#'
+else
+ MODULE_SPWD_TRUE='#'
+ MODULE_SPWD_FALSE=
+fi
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: result: $py_cv_module_spwd" >&5
+$as_echo "$py_cv_module_spwd" >&6; }
+
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module syslog" >&5
+$as_echo_n "checking for stdlib extension module syslog... " >&6; }
+ case $py_stdlib_not_available in #(
+ *syslog*) :
+ py_cv_module_syslog=n/a ;; #(
+ *) :
+
+ if true; then :
+ if test "$ac_cv_header_syslog_h" = yes; then :
+ py_cv_module_syslog=yes
+else
+ py_cv_module_syslog=missing
+fi
+else
+ py_cv_module_syslog=disabled
+
+fi
+
+ ;;
+esac
+ as_fn_append MODULE_BLOCK "MODULE_SYSLOG=$py_cv_module_syslog$as_nl"
+ if test "x$py_cv_module_syslog" = xyes; then :
+
+
+
+
+fi
+ if test "$py_cv_module_syslog" = yes; then
+ MODULE_SYSLOG_TRUE=
+ MODULE_SYSLOG_FALSE='#'
+else
+ MODULE_SYSLOG_TRUE='#'
+ MODULE_SYSLOG_FALSE=
+fi
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: result: $py_cv_module_syslog" >&5
+$as_echo "$py_cv_module_syslog" >&6; }
+
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module termios" >&5
+$as_echo_n "checking for stdlib extension module termios... " >&6; }
+ case $py_stdlib_not_available in #(
+ *termios*) :
+ py_cv_module_termios=n/a ;; #(
+ *) :
+
+ if true; then :
+ if test "$ac_cv_header_termios_h" = yes; then :
+ py_cv_module_termios=yes
+else
+ py_cv_module_termios=missing
+fi
+else
+ py_cv_module_termios=disabled
+
+fi
+
+ ;;
+esac
+ as_fn_append MODULE_BLOCK "MODULE_TERMIOS=$py_cv_module_termios$as_nl"
+ if test "x$py_cv_module_termios" = xyes; then :
+
+
+
+
+fi
+ if test "$py_cv_module_termios" = yes; then
+ MODULE_TERMIOS_TRUE=
+ MODULE_TERMIOS_FALSE='#'
+else
+ MODULE_TERMIOS_TRUE='#'
+ MODULE_TERMIOS_FALSE=
+fi
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: result: $py_cv_module_termios" >&5
+$as_echo "$py_cv_module_termios" >&6; }
+
+

{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for stdlib extension module pyexpat" >&5
$as_echo_n "checking for stdlib extension module pyexpat... " >&6; }
@@ -20428,14 +20633,34 @@ if test -z "${MODULE__DATETIME_TRUE}" && test -z "${MODULE__DATETIME_FALSE}"; th
as_fn_error $? "conditional \"MODULE__DATETIME\" was never defined.
Usually this means the macro was only invoked conditionally." "$LINENO" 5
fi
+if test -z "${MODULE_GRP_TRUE}" && test -z "${MODULE_GRP_FALSE}"; then
+ as_fn_error $? "conditional \"MODULE_GRP\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
if test -z "${MODULE_OSSAUDIODEV_TRUE}" && test -z "${MODULE_OSSAUDIODEV_FALSE}"; then
as_fn_error $? "conditional \"MODULE_OSSAUDIODEV\" was never defined.
Usually this means the macro was only invoked conditionally." "$LINENO" 5
fi
+if test -z "${MODULE_RESOURCE_TRUE}" && test -z "${MODULE_RESOURCE_FALSE}"; then
+ as_fn_error $? "conditional \"MODULE_RESOURCE\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
if test -z "${MODULE__SCPROXY_TRUE}" && test -z "${MODULE__SCPROXY_FALSE}"; then
as_fn_error $? "conditional \"MODULE__SCPROXY\" was never defined.
Usually this means the macro was only invoked conditionally." "$LINENO" 5
fi
+if test -z "${MODULE_SPWD_TRUE}" && test -z "${MODULE_SPWD_FALSE}"; then
+ as_fn_error $? "conditional \"MODULE_SPWD\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
+if test -z "${MODULE_SYSLOG_TRUE}" && test -z "${MODULE_SYSLOG_FALSE}"; then
+ as_fn_error $? "conditional \"MODULE_SYSLOG\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
+if test -z "${MODULE_TERMIOS_TRUE}" && test -z "${MODULE_TERMIOS_FALSE}"; then
+ as_fn_error $? "conditional \"MODULE_TERMIOS\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
if test -z "${MODULE_PYEXPAT_TRUE}" && test -z "${MODULE_PYEXPAT_FALSE}"; then
as_fn_error $? "conditional \"MODULE_PYEXPAT\" was never defined.
Usually this means the macro was only invoked conditionally." "$LINENO" 5
diff --git a/configure.ac b/configure.ac
index 6c24a278595a6..3d6233f31d9cb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2162,7 +2162,7 @@ sys/times.h sys/types.h sys/uio.h sys/un.h sys/utsname.h sys/wait.h pty.h \
libutil.h sys/resource.h netpacket/packet.h sysexits.h bluetooth.h \
linux/tipc.h linux/random.h spawn.h util.h alloca.h endian.h \
sys/endian.h sys/sysmacros.h linux/memfd.h linux/wait.h sys/memfd.h \
-sys/mman.h sys/eventfd.h linux/soundcard.h sys/soundcard.h)
+sys/mman.h sys/eventfd.h linux/soundcard.h sys/soundcard.h syslog.h)
AC_HEADER_DIRENT
AC_HEADER_MAJOR

@@ -3854,7 +3854,7 @@ AC_CHECK_FUNCS(alarm accept4 setitimer getitimer bind_textdomain_codeset chown \
explicit_memset faccessat fchmod fchmodat fchown fchownat \
fdwalk fexecve fdopendir fork fpathconf fstatat ftime ftruncate futimesat \
futimens futimes gai_strerror getentropy \
- getgrgid_r getgrnam_r \
+ getgrgid getgrgid_r getgrnam_r \
getgrouplist getgroups getlogin getloadavg getpeername getpgid getpid \
getpriority getresuid getresgid getpwent getpwnam_r getpwuid_r getspnam getspent getsid getwd \
if_nameindex \
@@ -6060,11 +6060,16 @@ dnl always enabled extension modules
PY_STDLIB_MOD_SIMPLE([_datetime], [], [$TIMEMODULE_LIB $LIBM])

dnl platform specific extensions
+PY_STDLIB_MOD([grp], [], [test "$ac_cv_func_getgrgid" = yes -o "$ac_cv_func_getgrgid_r" = yes])
PY_STDLIB_MOD([ossaudiodev],
[], [test "$ac_cv_header_linux_soundcard_h" = yes -o "$ac_cv_header_sys_soundcard_h" = yes])
+PY_STDLIB_MOD([resource], [], [test "$ac_cv_header_sys_resource_h" = yes])
PY_STDLIB_MOD([_scproxy],
[test "$ac_sys_system" = "Darwin"], [],
[], [-framework SystemConfiguration -framework CoreFoundation])
+PY_STDLIB_MOD([spwd], [], [test "$ac_cv_func_getspent" = yes -o "$ac_cv_func_getspnam" = yes])
+PY_STDLIB_MOD([syslog], [], [test "$ac_cv_header_syslog_h" = yes])
+PY_STDLIB_MOD([termios], [], [test "$ac_cv_header_termios_h" = yes])

dnl _elementtree loads libexpat via CAPI hook in pyexpat
PY_STDLIB_MOD([pyexpat], [], [], [$LIBEXPAT_CFLAGS], [$LIBEXPAT_LDFLAGS])
diff --git a/pyconfig.h.in b/pyconfig.h.in
index b8f63955e3139..afd3ebe35bc0d 100644
--- a/pyconfig.h.in
+++ b/pyconfig.h.in
@@ -441,6 +441,9 @@
/* Define to 1 if you have the `getentropy' function. */
#undef HAVE_GETENTROPY

+/* Define to 1 if you have the `getgrgid' function. */
+#undef HAVE_GETGRGID
+
/* Define to 1 if you have the `getgrgid_r' function. */
#undef HAVE_GETGRGID_R

@@ -1143,6 +1146,9 @@
/* Define to 1 if you have the <sysexits.h> header file. */
#undef HAVE_SYSEXITS_H

+/* Define to 1 if you have the <syslog.h> header file. */
+#undef HAVE_SYSLOG_H
+
/* Define to 1 if you have the <sys/audioio.h> header file. */
#undef HAVE_SYS_AUDIOIO_H

diff --git a/setup.py b/setup.py
index d01cc4f7169ec..9ff5969c29613 100644
--- a/setup.py
+++ b/setup.py
@@ -1046,16 +1046,8 @@ def detect_simple_extensions(self):
self.add(Extension('fcntl', ['fcntlmodule.c'],
libraries=libs))
# grp(3)
- if not VXWORKS:
- self.add(Extension('grp', ['grpmodule.c']))
- # spwd, shadow passwords
- if (self.config_h_vars.get('HAVE_GETSPNAM', False) or
- self.config_h_vars.get('HAVE_GETSPENT', False)):
- self.add(Extension('spwd', ['spwdmodule.c']))
- # AIX has shadow passwords, but access is not via getspent(), etc.
- # module support is not expected so it not 'missing'
- elif not AIX:
- self.missing.append('spwd')
+ self.addext(Extension('grp', ['grpmodule.c']))
+ self.addext(Extension('spwd', ['spwdmodule.c']))

# select(2); not on ancient System V
self.add(Extension('select', ['selectmodule.c']))
@@ -1065,7 +1057,7 @@ def detect_simple_extensions(self):

# Lance Ellinghaus's syslog module
# syslog daemon interface
- self.add(Extension('syslog', ['syslogmodule.c']))
+ self.addext(Extension('syslog', ['syslogmodule.c']))

# Python interface to subinterpreter C-API.
self.add(Extension('_xxsubinterpreters', ['_xxsubinterpretersmodule.c']))
@@ -1375,15 +1367,10 @@ def detect_sqlite(self):

def detect_platform_specific_exts(self):
# Unix-only modules
- if not MS_WINDOWS:
- if not VXWORKS:
- # Steen Lumholt's termios module
- self.add(Extension('termios', ['termios.c']))
- # Jeremy Hylton's rlimit interface
- self.add(Extension('resource', ['resource.c']))
- else:
- self.missing.extend(['resource', 'termios'])
-
+ # Steen Lumholt's termios module
+ self.addext(Extension('termios', ['termios.c']))
+ # Jeremy Hylton's rlimit interface
+ self.addext(Extension('resource', ['resource.c']))
# linux/soundcard.h or sys/soundcard.h
self.addext(Extension('ossaudiodev', ['ossaudiodev.c']))


_______________________________________________
Python-checkins mailing list
Python-checkins@python.org
https://mail.python.org/mailman/listinfo/python-checkins