Re: [PATCH] Missing dep on Catalog.pm in meson rules - Mailing list pgsql-hackers

From Dagfinn Ilmari Mannsåker
Subject Re: [PATCH] Missing dep on Catalog.pm in meson rules
Date
Msg-id 871qibrrcd.fsf@wibble.ilmari.org
Whole thread Raw
In response to Re: [PATCH] Missing dep on Catalog.pm in meson rules  (Andres Freund <andres@anarazel.de>)
Responses Re: [PATCH] Missing dep on Catalog.pm in meson rules
List pgsql-hackers
Andres Freund <andres@anarazel.de> writes:

> Hi,
>
> On 2023-06-16 16:20:14 -0400, Andrew Dunstan wrote:
>> Unless I'm misunderstanding, this doesn't look terribly feasible to me. You
>> can only get at %INC by loading the module, which in many cases will have
>> side effects.
>
> I was envisioning using %INC after the use/require block - I don't think our
> scripts load additional modules after that point?

I was thinking of a module for writing depfile entries that would append
`values %INC` to the list of source files for each target specified by
the script.

>> And then you would also need to filter out things loaded that
>> are not our artefacts (e.g. Catalog.pm loads File::Compare).
>
> I don't think we would need to filter the output. This would just be for a
> build dependency file. I don't see a problem with rerunning genbki.pl et al after
> somebody updates File::Compare?

As long as mason doesn't object to dep files outside the source tree.
Otherwise, and option would be to pass in @SOURCE_ROOT@ and only include
`grep /^\Q$source_root\E\b/, values %INC` in the depfile.

> Greetings,
>
> Andres Freund

- ilmari



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Default client_connection_check_interval to 10s on supported systems
Next
From: "Tristan Partin"
Date:
Subject: Re: [PATCH] Missing dep on Catalog.pm in meson rules