Re: Coverage improvements of src/bin/pg_basebackup and pg_receivewal --endpos

From: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Coverage improvements of src/bin/pg_basebackup and pg_receivewal --endpos
Date: 2017-09-11 20:57:15
Message-ID: 49b529c1-0f44-d905-b33c-005ec0114d09@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dmitry Dolgov 2017-09-11 21:12:29 Re: [PATCH] Generic type subscripting
Previous Message Tom Lane 2017-09-11 20:28:01 Re: Re: proposal - using names as primary names of plpgsql function parameters instead $ based names