diff options
| author | Tom Lane | 2019-01-18 20:06:26 +0000 |
|---|---|---|
| committer | Tom Lane | 2019-01-18 20:06:26 +0000 |
| commit | 167ba6b1569d5a96d0452fbe708e866222d5b8a8 (patch) | |
| tree | 0905f92c091f6ea22263a384f61f97baba4471ed /configure.in | |
| parent | 9450b91cdf140e7becb5e9bb477da0c1f6cdbdb1 (diff) | |
Use our own getopt() on OpenBSD.
Recent OpenBSD (at least 5.9 and up) has a version of getopt(3)
that will not cope with the "-:" spec we use to accept double-dash
options in postgres.c and postmaster.c. Admittedly, that's a hack
because POSIX only requires getopt() to allow alphanumeric option
characters. I have no desire to find another way, however, so
let's just do what we were already doing on Solaris: force use
of our own src/port/getopt.c implementation.
In passing, improve some of the comments around said implementation.
Per buildfarm and local testing. Back-patch to all supported branches.
Discussion: https://postgr.es/m/30197.1547835700@sss.pgh.pa.us
Diffstat (limited to 'configure.in')
| -rw-r--r-- | configure.in | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/configure.in b/configure.in index bfb03f892d2..86fbdff8d44 100644 --- a/configure.in +++ b/configure.in @@ -1652,9 +1652,9 @@ else AC_LIBOBJ(getopt_long) fi -# Solaris' getopt() doesn't do what we want for long options, so always use -# our version on that platform. -if test "$PORTNAME" = "solaris"; then +# On OpenBSD and Solaris, getopt() doesn't do what we want for long options +# (i.e., allow '-' as a flag character), so use our version on those platforms. +if test "$PORTNAME" = "openbsd" -o "$PORTNAME" = "solaris"; then AC_LIBOBJ(getopt) fi |