Re: Meson build updates

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tristan Partin <tristan(at)neon(dot)tech>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Meson build updates
Date: 2023-06-12 17:20:38
Message-ID: 20230612172038.f2xf2cwlnhjldqng@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2023-06-12 11:54:42 -0500, Tristan Partin wrote:
> On Fri Jun 9, 2023 at 1:36 PM CDT, Andres Freund wrote:
> > The biggest change to make them more usable would be to properly reconfigure
> > when the contents of machine file change. IIRC configure is rerun, but the
> > changes aren't taken into account.
>
> I could not reproduce this. Perhaps you were testing with an older Meson
> where that was the case
>
> # meson.build
> project('mytest')
>
> myprog = find_program('myprog')
> message(myprog.full_path())
>
> test('dummy', find_program('echo'), args: [myprog.full_path()])
>
> # file.ini
> [binaries]
> myprog = '/usr/bin/python3'
>
> # CLI
> meson setup build
> meson test -C build
> sed -i 's/python3/python2/' file.ini
> meson test -C build

It's possible that it doesn't happen in all contexts. I just reproduced the
problem I had, changing

[binaries]
llvm-config = '/usr/bin/llvm-config-13'

to

[binaries]
llvm-config = '/usr/bin/llvm-config-14'

does not change which version is used in an existing build tree, but does
change what's used in a new build tree.

Same with e.g. changing the C compiler version in a machine file. That also
only takes effect in a new tree.

This is with meson HEAD, updated earlier today.

> > In the end, I also just don't see a meaningful benefit in forcing the use of
> > machine files.
>
> I think it is best to use patterns tools want you to use.

Sometimes. I'd perhaps have a different view if we weren't migrating from
autoconf, where overwriting binaries was trivially possible...

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2023-06-12 17:33:45 Re: pgsql: Fix search_path to a safe value during maintenance operations.
Previous Message Andres Freund 2023-06-12 17:10:33 Re: [RFC] building postgres with meson - v12