Fix our getopt_long's behavior for a command line argument of just "-".
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 23 Mar 2020 15:58:00 +0000 (11:58 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 23 Mar 2020 15:58:00 +0000 (11:58 -0400)
commit980a70b9763e330dab6d0eb994c96a007876e40e
tree0ac3c0b304aef20c27a28647d195e37db5368200
parentc81bd3b9a5e99315148f4f5714c32e633af30123
Fix our getopt_long's behavior for a command line argument of just "-".

src/port/getopt_long.c failed on such an argument, always seeing it
as an unrecognized switch.  This is unhelpful; better is to treat such
an item as a non-switch argument.  That behavior is what we find in
GNU's getopt_long(); it's what src/port/getopt.c does; and it is
required by POSIX for getopt(), which getopt_long() ought to be
generally a superset of.  Moreover, it's expected by ecpg, which
intends an argument of "-" to mean "read from stdin".  So fix it.

Also add some documentation about ecpg's behavior in this area, since
that was miserably underdocumented.  I had to reverse-engineer it
from the code.

Per bug #16304 from James Gray.  Back-patch to all supported branches,
since this has been broken forever.

Discussion: https://postgr.es/m/16304-c662b00a1322db7f@postgresql.org
doc/src/sgml/ref/ecpg-ref.sgml
src/port/getopt_long.c