forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit3a3c1b8
committed
From: Tom Lane <tgl@sss.pgh.pa.us>I see someone missed an ancient bit of shell-scripting lore:on some older shells, if your script's argument list is empty,then "$@" generates an empty-string word rather than no wordat all. You need to write ${1+"$@"} to get the latter behavior.(Read your shell man page to see exactly how that works,but it does the Right Thing on every Bourne shell.)In particular, pg_dumpall fails when invoked without any switcheson HPUX 9.*, because pg_dump gets an empty-string argument that itthinks is the name of the database to dump. I expect this bugalso affects some other OSes, but couldn't tell you just which ones.Patch attached.
1 parentbd029bc commit3a3c1b8
1 file changed
+1
-1
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | | - | |
| 61 | + | |
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
| |||
0 commit comments
Comments
(0)