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 18:07:17 |
Message-ID: | 9db0d2fe-de8a-727a-94ac-799fb0ee6ecb@2ndQuadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 3/1/19 12:56 PM, Andrew Dunstan wrote:
> 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 ['`]
... and on Windows the result is radically different:
# Failed test 'pg_dumpall: option --exclude-database requires an argument: matches'
# at t/001_basic.pl line 154.
# 'H:\prog\bf\root\HEAD\pgsql.build\tmp_install\bin\pg_dumpall.EXE: option requires an argument -- exclude-database
# Try "pg_dumpall --help" for more information.
# '
# doesn't match '(?^:pg_dumpall\:\ option\ \'\-\-exclude\-database\'\ requires\ an\ argument)'
I'm somewhat inclined just to abandon the test. It's at best of marginal
use.
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2019-03-01 18:44:46 | pgsql: Store tuples for EvalPlanQual in slots, rather than as HeapTuple |
Previous Message | Andrew Dunstan | 2019-03-01 17:56:32 | Re: pgsql: Add --exclude-database option to pg_dumpall |