Re: Improve prep_buildtree

From: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Improve prep_buildtree
Date: 2025-07-30 10:58:43
Message-ID: CAN55FZ06Gcx1o3ykCNMrGn9Eyn+yGNwCkxaddDyg80oxEWSv3g@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Wed, 30 Jul 2025 at 12:32, Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
>
> On 30.07.25 10:28, Nazir Bilal Yavuz wrote:
> > On Wed, 30 Jul 2025 at 11:07, Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> >>
> >> To fix this, I first tried to devise a way to detect whether a given
> >> directory is a build directory. But that seemed pretty complicated.
> >> Instead, I chose the simpler solution that we just enumerate the source
> >> subdirectories that we know about (config, contrib, doc, src). That way
> >> we can also remove the special handling to exclude the .git directory
> >> and make the find command a bit simpler.
> >
> > Meson looks for ${builddir}/meson-private/coredata.dat file to
> > understand if the directory is a build directory [1]. I am just
> > wondering if you tried this and found it complicated or did you try
> > something else.
>
> I tried things along this line, but it's not easy to have a shell "find"
> command handle that kind of logic.

That makes sense. I noticed one difference, patched prep_buildtree
does not create a symbolic link for the top Makefile in the builddir.

--
Regards,
Nazir Bilal Yavuz
Microsoft

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Japin Li 2025-07-30 11:07:38 Re: [WIP]Vertical Clustered Index (columnar store extension) - take2
Previous Message Dagfinn Ilmari Mannsåker 2025-07-30 10:17:55 Re: Fix tab completion in v18 for ALTER DATABASE/USER/ROLE ... RESET