Hi, On 2023-01-04 23:17:30 +0100, Peter Eisentraut wrote: > I meant the latter, which I see is already in there, but it doesn't actually > fully work. It only looks at the subdirectory (like "lib"), not the whole > path (like "/usr/local/pgsql/lib"). With the attached patch I have it > working and I get the same installation layout from both build systems.
Oh, oops. I tested this at some point, but I guess I over-simplified it at some point. Then I have zero objections to this. One question below though. > dir_data = get_option('datadir') > -if not (dir_data.contains('pgsql') or dir_data.contains('postgres')) > +if not ((dir_prefix/dir_data).contains('pgsql') or > (dir_prefix/dir_data).contains('postgres')) > dir_data = dir_data / pkg > endif Hm. Perhaps we should just test once whether prefix contains pgsql/postgres, and then just otherwise leave the test as is? There afaict can't be a dir_prefix/dir_* that matches postgres/pgsql that won't also match either of the components. Greetings, Andres Freund