Skip to content
Snippets Groups Projects
Select Git revision
  • benchmark-tools
  • postgres-lambda
  • master default
  • REL9_4_25
  • REL9_5_20
  • REL9_6_16
  • REL_10_11
  • REL_11_6
  • REL_12_1
  • REL_12_0
  • REL_12_RC1
  • REL_12_BETA4
  • REL9_4_24
  • REL9_5_19
  • REL9_6_15
  • REL_10_10
  • REL_11_5
  • REL_12_BETA3
  • REL9_4_23
  • REL9_5_18
  • REL9_6_14
  • REL_10_9
  • REL_11_4
23 results

src

  • Clone with SSH
  • Clone with HTTPS
  • user avatar
    Tom Lane authored
    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 be76a6d3 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
    dfbad3f4
    History