Re: pgsql: Add TAP test to automate the equivalent of check_guc

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Christoph Berg <myon(at)debian(dot)org>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add TAP test to automate the equivalent of check_guc
Date: 2022-02-11 14:59:55
Message-ID: 2023925.1644591595@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Christoph Berg <myon(at)debian(dot)org> writes:
> this test is failing at Debian package compile time:
> Could not open /usr/share/postgresql/15/postgresql.conf.sample: No such file or directory at t/003_check_guc.pl line 47.

> So it's trying to read from /usr/share/postgresql which doesn't exist
> yet at build time.

> The relevant part of the test is this:

> # Find the location of postgresql.conf.sample, based on the information
> # provided by pg_config.
> my $sample_file =
> $node->config_data('--sharedir') . '/postgresql.conf.sample';

This seems like a pretty bad idea even if it weren't failing outright.
We should be examining the version of the file that's in the source
tree; the one in the installation tree might have version-skew
problems, if you've not yet done "make install".

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Justin Pryzby 2022-02-11 15:35:49 Re: pgsql: Add TAP test to automate the equivalent of check_guc
Previous Message Robert Haas 2022-02-11 14:49:40 pgsql: pg_basebackup: Allow client-side LZ4 (de)compression.

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2022-02-11 15:01:07 Re: refactoring basebackup.c
Previous Message Peter Eisentraut 2022-02-11 14:28:19 Re: Synchronizing slots from primary to standby