Re: Non-portable shell code in pg_upgrade tap tests - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Non-portable shell code in pg_upgrade tap tests
Date
Msg-id 17833.1532096747@sss.pgh.pa.us
Whole thread Raw
In response to Non-portable shell code in pg_upgrade tap tests  (Victor Wagner <vitus@wagner.pp.ru>)
Responses Re: Non-portable shell code in pg_upgrade tap tests  (Victor Wagner <vitus@wagner.pp.ru>)
List pgsql-hackers
Victor Wagner <vitus@wagner.pp.ru> writes:
> I've discovered that in the branch REL_11_STABLE there is shell script
> src/bin/pg_upgrade/test.sh which doesn't work under Solaris 10.
> (it uses $(command) syntax with is not compatible with original
> Solaris /bin/sh)

OK ...

> It is quite easy to replace $() syntax with backticks. Backticks are
> not nestable and considered unsafe by modern shell scripting style
> guides, but they do work with older shells.

Please send a patch.  Most of us do not have access to old shells
we could test this on.  (The oldest machine I have, and it's old
enough to vote, does run that script ... I doubt very many other
developers have anything older.)

            regards, tom lane


pgsql-hackers by date:

Previous
From: Victor Wagner
Date:
Subject: Non-portable shell code in pg_upgrade tap tests
Next
From: Tom Lane
Date:
Subject: Re: Fw: Windows 10 got stuck with PostgreSQL at starting up. Adding delay lets it avoid.