From 0f19d0f12fa235c6951ff431bd60da594356a1a6 Mon Sep 17 00:00:00 2001
From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon, 31 Aug 2015 01:36:46 -0400
Subject: [PATCH] Remove long-dead support for platforms without sig_atomic_t.
C89 requires <signal.h> to define sig_atomic_t, and there is no evidence
in the buildfarm that any supported platforms don't comply. Remove the
configure test to stop wasting build cycles on a purely historical issue.
(Once upon a time, we cared about supporting C89-compliant compilers on
machines with pre-C89 system headers, but that use-case has been dead for
quite a few years.)
I have some other fixes planned in this area, but let's start with this
to see if the buildfarm produces any surprising results.
---
configure | 14 --------------
configure.in | 4 ----
src/include/c.h | 5 -----
src/include/pg_config.h.in | 3 ---
src/include/pg_config.h.win32 | 3 ---
5 files changed, 29 deletions(-)
diff --git a/configure b/configure
index 0bed81c2e4a..0376a7ca919 100755
--- a/configure
+++ b/configure
@@ -14151,20 +14151,6 @@ _ACEOF
fi
-# We also check for sig_atomic_t, which *should* be defined per ANSI
-# C, but is missing on some old platforms.
-ac_fn_c_check_type "$LINENO" "sig_atomic_t" "ac_cv_type_sig_atomic_t" "#include <signal.h>
-"
-if test "x$ac_cv_type_sig_atomic_t" = xyes; then :
-
-cat >>confdefs.h <<_ACEOF
-#define HAVE_SIG_ATOMIC_T 1
-_ACEOF
-
-
-fi
-
-
# Check for extensions offering the integer scalar type __int128.
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for __int128" >&5
$as_echo_n "checking for __int128... " >&6; }
diff --git a/configure.in b/configure.in
index a28f9ddb252..5f16530fca8 100644
--- a/configure.in
+++ b/configure.in
@@ -1831,10 +1831,6 @@ AC_DEFINE_UNQUOTED(MAXIMUM_ALIGNOF, $MAX_ALIGNOF, [Define as the maximum alignme
AC_CHECK_TYPES([int8, uint8, int64, uint64], [], [],
[#include <stdio.h>])
-# We also check for sig_atomic_t, which *should* be defined per ANSI
-# C, but is missing on some old platforms.
-AC_CHECK_TYPES(sig_atomic_t, [], [], [#include <signal.h>])
-
# Check for extensions offering the integer scalar type __int128.
PGAC_TYPE_128BIT_INT
diff --git a/src/include/c.h b/src/include/c.h
index b719eb95e14..f5da4676c6f 100644
--- a/src/include/c.h
+++ b/src/include/c.h
@@ -334,11 +334,6 @@ typedef unsigned PG_INT128_TYPE uint128;
#define HAVE_INT64_TIMESTAMP
#endif
-/* sig_atomic_t is required by ANSI C, but may be missing on old platforms */
-#ifndef HAVE_SIG_ATOMIC_T
-typedef int sig_atomic_t;
-#endif
-
/*
* Size
* Size of any memory resident object, as returned by sizeof.
diff --git a/src/include/pg_config.h.in b/src/include/pg_config.h.in
index 8873dccd1e2..1dcc9a9ee7a 100644
--- a/src/include/pg_config.h.in
+++ b/src/include/pg_config.h.in
@@ -449,9 +449,6 @@
/* Define to 1 if you have sigsetjmp(). */
#undef HAVE_SIGSETJMP
-/* Define to 1 if the system has the type `sig_atomic_t'. */
-#undef HAVE_SIG_ATOMIC_T
-
/* Define to 1 if you have the `snprintf' function. */
#undef HAVE_SNPRINTF
diff --git a/src/include/pg_config.h.win32 b/src/include/pg_config.h.win32
index ad61392841e..bf69ef5bdeb 100644
--- a/src/include/pg_config.h.win32
+++ b/src/include/pg_config.h.win32
@@ -322,9 +322,6 @@
/* Define to 1 if you have sigsetjmp(). */
/* #undef HAVE_SIGSETJMP */
-/* Define to 1 if the system has the type `sig_atomic_t'. */
-#define HAVE_SIG_ATOMIC_T 1
-
/* Define to 1 if you have the `snprintf' function. */
/* #undef HAVE_SNPRINTF */
--
GitLab