postgres.git / summary / log / commit / refs

commit    0268d21e5d3c732bf5543d68a6d870e4eee7e673
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Mon Mar 17 01:55:02 2014 +0000
First released in: 9.4.0, 9.2.8, 9.3.4

    Fix unportable shell-script syntax in pg_upgrade's test.sh.
    
    I discovered the hard way that on some old shells, the locution
        FOO=""   unset FOO
    does not behave the same as
        FOO="";  unset FOO
    and in fact leaves FOO set to an empty string.  test.sh was inconsistently
    spelling it different ways on adjacent lines.
    
    This got broken relatively recently, in commit c737a2e56, so the lack of
    field reports to date doesn't represent a lot of evidence that the problem
    is rare.


 contrib/pg_upgrade/test.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contrib/pg_upgrade/test.sh b/contrib/pg_upgrade/test.sh
index 1ef80edbcbb..baa7d4748b6 100644
--- a/contrib/pg_upgrade/test.sh
+++ b/contrib/pg_upgrade/test.sh
@@ -83,10 +83,10 @@ mkdir "$logdir"
 PGDATABASE="";        unset PGDATABASE
 PGUSER="";            unset PGUSER
 PGSERVICE="";         unset PGSERVICE
-PGSSLMODE=""          unset PGSSLMODE
+PGSSLMODE="";         unset PGSSLMODE
 PGREQUIRESSL="";      unset PGREQUIRESSL
 PGCONNECT_TIMEOUT=""; unset PGCONNECT_TIMEOUT
-PGHOST=""             unset PGHOST
+PGHOST="";            unset PGHOST
 PGHOSTADDR="";        unset PGHOSTADDR
 
 # Select a non-conflicting port number, similarly to pg_regress.c


[parent: 2861e8e9cb7b]