Re: [HACKERS] Coverage improvements of src/bin/pg_basebackup andpg_receivewal --endpos - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: [HACKERS] Coverage improvements of src/bin/pg_basebackup andpg_receivewal --endpos
Date
Msg-id 49b529c1-0f44-d905-b33c-005ec0114d09@2ndquadrant.com
Whole thread Raw
In response to Re: [HACKERS] Coverage improvements of src/bin/pg_basebackup andpg_receivewal --endpos  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: [HACKERS] Coverage improvements of src/bin/pg_basebackup andpg_receivewal --endpos
List pgsql-hackers
On 9/6/17 00:54, Michael Paquier wrote:
>> - I think the tests will fail if libz support is not built.  Again,
>> pg_basebackup doesn't have tests for that.  So maybe omit that and
>> address it later.
> 
> Let's address it now. This can be countered by querying pg_config()
> before running the command of pg_receivexwal which uses --compress.

I have committed this patch, after a perltidy run, but the way the libz
detection was implemented was a bit too hackish for me, so I have
omitted that for now.  I think a more robust way would be to parse
Makefile.global, perhaps by a function in TestLib, so it can be reused
in other tests.  (Even more robust would be to write out an actual Perl
file with configuration information somewhere, but maybe we don't need
to go there yet.)  Or maybe have the respective make variables exported
when make check is called (could be included in the prove_check
variable?).  Anyway, some more pondering could lead to a more general
solution.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] Re: proposal - using names as primary names of plpgsql function parameters instead $ based names
Next
From: Dmitry Dolgov
Date:
Subject: Re: [HACKERS] [PATCH] Generic type subscripting