diff options
author | Thomas Munro <tmunro@postgresql.org> | 2022-08-05 09:21:51 +1200 |
---|---|---|
committer | Thomas Munro <tmunro@postgresql.org> | 2022-08-05 09:21:51 +1200 |
commit | adeef67834cc3db14c36430c1421cf4ed8a809d3 (patch) | |
tree | 2f5b03f941cbe18f2d7fcb7940dd9b436020d58e | |
parent | 098f4d813b69b4f88943afa777ae4bba5166f803 (diff) | |
download | postgresql-adeef67834cc3db14c36430c1421cf4ed8a809d3.tar.gz postgresql-adeef67834cc3db14c36430c1421cf4ed8a809d3.zip |
Remove configure probe for setsid.
setsid() is in SUSv2 and all targeted Unix systems have it. Retain a
HAVE_SETSID macro, defined on Unix only. That's easier to understand
than !defined(WIN32), for the optional code it governs.
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://postgr.es/m/CA+hUKGJ3LHeP9w5Fgzdr4G8AnEtJ=z=p6hGDEm4qYGEUX5B6fQ@mail.gmail.com
-rwxr-xr-x | configure | 2 | ||||
-rw-r--r-- | configure.ac | 1 | ||||
-rw-r--r-- | src/include/pg_config.h.in | 3 | ||||
-rw-r--r-- | src/include/port.h | 1 | ||||
-rw-r--r-- | src/tools/msvc/Solution.pm | 1 |
5 files changed, 2 insertions, 6 deletions
diff --git a/configure b/configure index 8cbd03fe0ad..232f19a0cd2 100755 --- a/configure +++ b/configure @@ -16039,7 +16039,7 @@ fi LIBS_including_readline="$LIBS" LIBS=`echo "$LIBS" | sed -e 's/-ledit//g' -e 's/-lreadline//g'` -for ac_func in backtrace_symbols clock_gettime copyfile fdatasync getifaddrs getpeerucred inet_pton kqueue mbstowcs_l memset_s poll posix_fallocate ppoll pthread_is_threaded_np readlink readv setproctitle setproctitle_fast setsid strchrnul strsignal symlink syncfs sync_file_range uselocale wcstombs_l writev +for ac_func in backtrace_symbols clock_gettime copyfile fdatasync getifaddrs getpeerucred inet_pton kqueue mbstowcs_l memset_s poll posix_fallocate ppoll pthread_is_threaded_np readlink readv setproctitle setproctitle_fast strchrnul strsignal symlink syncfs sync_file_range uselocale wcstombs_l writev do : as_ac_var=`$as_echo "ac_cv_func_$ac_func" | $as_tr_sh` ac_fn_c_check_func "$LINENO" "$ac_func" "$as_ac_var" diff --git a/configure.ac b/configure.ac index 9ef2a373f28..33dffe0ce55 100644 --- a/configure.ac +++ b/configure.ac @@ -1809,7 +1809,6 @@ AC_CHECK_FUNCS(m4_normalize([ readv setproctitle setproctitle_fast - setsid strchrnul strsignal symlink diff --git a/src/include/pg_config.h.in b/src/include/pg_config.h.in index 5015ac6a3ed..120eacfb200 100644 --- a/src/include/pg_config.h.in +++ b/src/include/pg_config.h.in @@ -469,9 +469,6 @@ /* Define to 1 if you have the `setproctitle_fast' function. */ #undef HAVE_SETPROCTITLE_FAST -/* Define to 1 if you have the `setsid' function. */ -#undef HAVE_SETSID - /* Define to 1 if the system has the type `socklen_t'. */ #undef HAVE_SOCKLEN_T diff --git a/src/include/port.h b/src/include/port.h index a8598532c01..7a1a02f6598 100644 --- a/src/include/port.h +++ b/src/include/port.h @@ -525,6 +525,7 @@ extern bool wait_result_is_any_signal(int exit_status, bool include_command_not_ */ #ifndef WIN32 #define HAVE_GETRLIMIT 1 +#define HAVE_SETSID 1 #define HAVE_SHM_OPEN 1 #endif diff --git a/src/tools/msvc/Solution.pm b/src/tools/msvc/Solution.pm index 5499498137c..a9680dc8630 100644 --- a/src/tools/msvc/Solution.pm +++ b/src/tools/msvc/Solution.pm @@ -350,7 +350,6 @@ sub GenerateFiles HAVE_SETENV => undef, HAVE_SETPROCTITLE => undef, HAVE_SETPROCTITLE_FAST => undef, - HAVE_SETSID => undef, HAVE_SOCKLEN_T => 1, HAVE_SPINLOCKS => 1, HAVE_STDBOOL_H => 1, |