]> granicus.if.org Git - postgresql/commit
Further portability hacking in pg_upgrade's test script.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Jul 2018 19:40:51 +0000 (15:40 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Jul 2018 19:40:51 +0000 (15:40 -0400)
commit04e3492c5c46822c756f575d6105a8a6ff76d4f2
treeac87ede944f3ea6061e09783938a0c25cafa6a88
parent5e5d131202c3494f4360f001b4805c8944e9600c
Further portability hacking in pg_upgrade's test script.

I blew the dust off a Bourne shell (file date 1996, yea verily) and
tried to run test.sh with it.  It mostly worked, but I found that the
temp-directory creation code introduced by commit be76a6d39 was not
compatible, for a couple of reasons: this shell thinks "set -e" should
force an exit if a command within backticks fails, and it also thinks code
within braces should be executed by a sub-shell, meaning that variable
settings don't propagate back up to the parent shell.  In view of Victor
Wagner's report that Solaris is still using pre-POSIX shells, seems like
we oughta make this case work.  It's not like the code is any less
idiomatic this way; the prior coding technique appeared nowhere else.

(There is a remaining bash-ism here, which is that $RANDOM doesn't do
what the code hopes in non-bash shells.  But the use of $$ elsewhere in
that path should be enough to ensure uniqueness and some amount of
randomness, so I think it's okay as-is.)

Back-patch to all supported branches, as the previous commit was.

Discussion: https://postgr.es/m/20180720153820.69e9ae6c@fafnir.local.vm
src/bin/pg_upgrade/test.sh