Re: pgsql: Add --exclude-database option to pg_dumpall

From: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add --exclude-database option to pg_dumpall
Date: 2019-03-01 17:56:32
Message-ID: 096370d7-5981-8dae-c47b-a1d3af33c3c5@2ndQuadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


On 3/1/19 12:04 PM, Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>> Add --exclude-database option to pg_dumpall
> Buildfarm seems to think you got the test case wrong ...
>
>

Hmm, Of course I tested this. It's passing on Linux boxes like crake and
prion, but failing on OSX and FBSD boxes.

I guess that's what we have a buildfarm for :-)

here's the error:

# Failed test 'pg_dumpall: option --exclude-database requires an argument: matches'
# at t/001_basic.pl line 154.
# 'pg_dumpall: option `--exclude-database' requires an argument
# Try "pg_dumpall --help" for more information.
# '
# doesn't match '(?^:pg_dumpall\:\ option\ \'\-\-exclude\-database\'\ requires\ an\ argument)'

Why is that backtick there? That looks like the culprit, coming from the
GetOpt implementation presumably.

I guess I can replace the first quote with a class containing ['`]

cheers

andrew

--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2019-03-01 18:07:17 Re: pgsql: Add --exclude-database option to pg_dumpall
Previous Message Tom Lane 2019-03-01 17:04:53 Re: pgsql: Add --exclude-database option to pg_dumpall