From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Broken make dependency somewhere near win32ver.o? |
Date: | 2022-03-22 15:30:18 |
Message-ID: | 20220322153018.ijuh3adub4kncihj@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2022-03-22 18:09:08 +1300, Thomas Munro wrote:
> On Tue, Mar 22, 2022 at 4:14 PM Andres Freund <andres(at)anarazel(dot)de> wrote:
> > The problem looks to be that pg_dumpall doesn't have a dependency on OBJs,
> > which in turn is what contains the dependency on WIN32RES, which in turn
> > contains win32ver.o. So the build succeeds if pg_dump/restores's dependencies
> > are built first, but not if pg_dumpall starts to be built before that...
> >
> > Seems we just need to add $(WIN32RES) to pg_dumpall: ?
>
> Ah, yeah, that looks right. I don't currently have a mingw setup to
> test, but clearly $(WIN32RES) is passed to $(CC) despite not being
> listed as a dependency.
Pushed a fix for that. Ended up doing it for all branches, although I was
debating with myself about doing so.
I did a quick search and didn't find other cases of this problem.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2022-03-22 15:37:57 | Re: refactoring basebackup.c (zstd workers) |
Previous Message | Andrew Dunstan | 2022-03-22 15:28:46 | Re: SQL/JSON: JSON_TABLE |