RE: Can't compile PG 17 (master) from git under Msys2 autoconf - Mailing list pgsql-hackers

From Regina Obe
Subject RE: Can't compile PG 17 (master) from git under Msys2 autoconf
Date
Msg-id 001b01da86e2$79dc69f0$6d953dd0$@pcorp.us
Whole thread Raw
In response to Re: Can't compile PG 17 (master) from git under Msys2 autoconf  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
List pgsql-hackers
> Hi Regina,
>
> On 2024-Mar-27, Regina Obe wrote:
>
> > The error is
> >
> > rm -f '../../src/include/storage/lwlocknames.h'
> > cp -pR ../../backend/storage/lmgr/lwlocknames.h
> > '../../src/include/storage/lwlocknames.h'
> > cp: cannot stat '../../backend/storage/lmgr/lwlocknames.h': No such
> > file or directory
> > make[1]: *** [Makefile:143: ../../src/include/storage/lwlocknames.h]
> > Error 1
> > make[1]: Leaving directory '/projects/postgresql/postgresql-
> git/src/backend'
> > make: *** [../../src/Makefile.global:382: submake-generated-headers]
> > Error 2
>
> Hmm, I changed these rules again in commit da952b415f44, maybe your
> problem is with that one?  I wonder if changing the references to
> ../include/storage/lwlocklist.h to $(topdir)/src/include/storage/lwlocklist.h
> (and similar things in
> src/backend/storage/lmgr/Makefile) would fix it.
>
> Do you run configure in the source directory or a separate build one?
>
> --
> Álvaro Herrera        Breisgau, Deutschland  —
> https://www.EnterpriseDB.com/
> "If it is not right, do not do it.
> If it is not true, do not say it." (Marcus Aurelius, Meditations)

I tried the change
> ../include/storage/lwlocklist.h to $(top_builddir)/src/include/storage/lwlocklist.h

I assume you meant that instead of $(topdir)

But nah that made no difference. Your change was already in my patched version so isn't causing any issues.

But as stated, rolling back this change in src/backend/Makefile in  0a475f01a4a (November 6th commit) makes it work for
me.

$(top_builddir)/src/include/storage/lwlocknames.h: storage/lmgr/lwlocknames.h
-   prereqdir=`cd '$(dir $<)' >/dev/null && pwd` && \
-     cd '$(dir $@)' && rm -f $(notdir $@) && \
-     $(LN_S) "$$prereqdir/$(notdir $<)" .
+   rm -f '$@'
+   $(LN_S) ../../backend/$< '$@'

 $(top_builddir)/src/include/utils/wait_event_types.h: utils/activity/wait_event_types.h
-   prereqdir=`cd '$(dir $<)' >/dev/null && pwd` && \
-     cd '$(dir $@)' && rm -f $(notdir $@) && \
-     $(LN_S) "$$prereqdir/$(notdir $<)" .
+   rm -f '$@'
+   $(LN_S) ../../backend/$< '$@'




pgsql-hackers by date:

Previous
From: Jelte Fennema-Nio
Date:
Subject: [MASSMAIL]Speed up clean meson builds by ~25%
Next
From: Jeff Davis
Date:
Subject: Re: LogwrtResult contended spinlock