Re: Fix fmgroids.h not regenerated after "clean" (not "clean dist") on Windows - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: Fix fmgroids.h not regenerated after "clean" (not "clean dist") on Windows
Date
Msg-id CABUevEy5391nqDyYwS=Dqk4jO8qqvv9crBWAQ-LEZNKr8n5zUw@mail.gmail.com
Whole thread Raw
In response to Fix fmgroids.h not regenerated after "clean" (not "clean dist") on Windows  (Craig Ringer <craig@2ndquadrant.com>)
List pgsql-hackers
On Wed, Dec 12, 2012 at 8:07 AM, Craig Ringer <craig@2ndquadrant.com> wrote:
> Hi all
>
> There's an issue with MSVC builds on Windows where clean.bat deletes
> src\include\utils\fmgroids.h (as it should) but build.pl doesn't
> re-create it reliably.
>
> It's created fine on the first build because Gen_fmgrtab.pl is called if
> src\backend\utils\fmgrtab.c is missing, which it is on first build.
> Generating it also generates src\backend\utils\fmgroids.h, which is
> copied to src\include. However, this copy isn't repeated after clean
> deletes src\include\utils\fmgroids.h .
>
> "clean dist" is fine, since it deletes fmgrtab.c too, causing the whole
> thing to be re-generated.
>
> The attached patch fixes the issue.

Looks good to me. Applied and backpatched to 9.2 - the logic appears
slightly different before that.


--Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/



pgsql-hackers by date:

Previous
From: Jan Wieck
Date:
Subject: Re: MySQL search query is not executing in Postgres DB
Next
From: Andres Freund
Date:
Subject: XLByte* usage