On 30.01.23 18:42, Andres Freund wrote:
On 2023-01-30 08:37:42 +0100, Peter Eisentraut wrote:
One open issue (IMO) with the meson build system is that it installs the
test modules under src/test/modules/ as part of a normal installation. This
is because there is no way to set up up the build system to install extra
things only when told. I think we still need a way to disable this somehow,
so that building a production installation doesn't end up with a bunch of
extra files.
The attached simple patch is a starting point for discussion. It just
disables the subdirectory src/test/modules/ based on some Boolean setting.
This could be some new top-level option, or maybe part of PG_TEST_EXTRA, or
something else? With this, I get an identical set of installed files from
meson. I imagine this option would be false by default and developers would
enable it.
Bilal, with a bit of help by me, worked on an alternative approach to
this. It's a lot more verbose in the initial change, but wouldn't increase the
amount of work/lines for new test modules. The main advantage is that we
wouldn't have disable the modules by default, which I think would be quite
likely to result in plenty people not running the tests.
Sending a link instead of attaching, in case you already registered a cfbot
entry:
https://github.com/anarazel/postgres/commit/d1d192a860da39af9aa63d7edf643eed0eeee7c4
Probably worth adding an install-test-modules target for manual use.
Looks like a good idea. I'm happy to proceed along that line.
