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
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 |