aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2017-03-07 11:36:35 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2017-03-07 11:36:42 -0500
commitef2662394455578f6c57e99a7896c69bdd9fbd74 (patch)
tree3b6547a0ff7489ee60cdd62a02962d98b206fb62 /src
parentaa56671836e6f80dc3bdc917dfc860ac10b822be (diff)
downloadpostgresql-ef2662394455578f6c57e99a7896c69bdd9fbd74.tar.gz
postgresql-ef2662394455578f6c57e99a7896c69bdd9fbd74.zip
Fix pgbench's failure to honor the documented long-form option "--builtin".
Not only did it not accept --builtin as a synonym for -b, but what it did accept as a synonym was --tpc-b (huh?), which it got even further wrong by marking as no_argument, so that if you did try that you got a core dump. I suppose this is leftover from some early design for the new switches added by commit 8bea3d221, but it's still pretty sloppy work. Per bug #14580 from Stepan Pesternikov. Back-patch to 9.6 where the error was introduced. Report: https://postgr.es/m/20170307123347.25054.73207@wrigleys.postgresql.org
Diffstat (limited to 'src')
-rw-r--r--src/bin/pgbench/pgbench.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
index f6cb5d4b10a..78f1e6b1e32 100644
--- a/src/bin/pgbench/pgbench.c
+++ b/src/bin/pgbench/pgbench.c
@@ -3610,7 +3610,7 @@ main(int argc, char **argv)
{
static struct option long_options[] = {
/* systematic long/short named options */
- {"tpc-b", no_argument, NULL, 'b'},
+ {"builtin", required_argument, NULL, 'b'},
{"client", required_argument, NULL, 'c'},
{"connect", no_argument, NULL, 'C'},
{"debug", no_argument, NULL, 'd'},