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