Re: Fix fmgroids.h not regenerated after "clean" (not "clean dist") on Windows

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Craig Ringer <craig(at)2ndquadrant(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix fmgroids.h not regenerated after "clean" (not "clean dist") on Windows
Date: 2012-12-16 14:00:16
Message-ID: CABUevEy5391nqDyYwS=Dqk4jO8qqvv9crBWAQ-LEZNKr8n5zUw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Dec 12, 2012 at 8:07 AM, Craig Ringer <craig(at)2ndquadrant(dot)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 Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2012-12-16 14:16:25 XLByte* usage
Previous Message Jan Wieck 2012-12-16 13:21:27 Re: MySQL search query is not executing in Postgres DB