pgsql: Remove dead setenv, unsetenv replacement code.

From: Thomas Munro <tmunro(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Remove dead setenv, unsetenv replacement code.
Date: 2022-08-04 21:59:42
Message-ID: E1oJisY-000Kmx-0a@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove dead setenv, unsetenv replacement code.

setenv() and unsetenv() are in SUSv3 and targeted Unix systems have
them. We still need special code for these on Windows, but that doesn't
require a configure probe.

This marks the first time we require a SUSv3 (POSIX.1-2001) facility
(rather than SUSv2). The replacement code removed here was not needed
on any targeted system or any known non-EOL'd Unix system, and was
therefore dead and untested.

No need for vestigial HAVE_SETENV and HAVE_UNSETENV macros, because we
provide a replacement for Windows, and we didn't previously test the
macros.

Reviewed-by: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Reviewed-by: Greg Stark <stark(at)mit(dot)edu>
Reviewed-by: Robert Haas <robertmhaas(at)gmail(dot)com>
Reviewed-by: Andres Freund <andres(at)anarazel(dot)de>
Discussion: https://postgr.es/m/CA+hUKGJ3LHeP9w5Fgzdr4G8AnEtJ=z=p6hGDEm4qYGEUX5B6fQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/71f5dc6dfb3de50de28ddde53793540c2fa98b1f

Modified Files
--------------
configure | 43 ------------------------------
configure.ac | 13 ----------
src/include/pg_config.h.in | 6 -----
src/include/port.h | 8 ------
src/port/setenv.c | 48 ----------------------------------
src/port/unsetenv.c | 65 ----------------------------------------------
src/tools/msvc/Solution.pm | 2 --
7 files changed, 185 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message David Rowley 2022-08-04 22:14:27 pgsql: Fix failure to set correct operator in window run condition
Previous Message Robert Haas 2022-08-04 19:30:33 pgsql: Revert recent changes to 002_pg_upgrade.pl.