From: | "Magnus Hagander" <mha(at)sollentuna(dot)net> |
---|---|
To: | "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>, "PostgreSQL-patches" <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Win32 VERSIONINFO fix |
Date: | 2004-10-06 14:27:16 |
Message-ID: | 6BCB9D8A16AC4241919521715F4D8BCE475E44@algol.sollentuna.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
> Unfortunately some re-jigging of Magnus' Win32 version patch
> broke the build by introducing some un-escaped double quotes
> into sed commands.
> The attached patch escapes those quotes without removing the
> re-jigging that was done.
Hmm, this approach seems to make it even less readable :-)
If I understood it right Bruce added the double quotes to the Makefiles
because he thought having a text string there without quotes was bad.
But having to escape them there is surely worse!
The original patch added the quotes in the sed rule. If we want to keep
the quotes in the Makefile:s, we should proably try to fix this problem
at the sed rule, not in every Makefile. If we can't, then my vote is to
go back to the orignial patch and remove the quotes from the Makefiles
completely.
I'm busy with other stuff right now, but I'll take a look at it as soon
as i can unless someone has already fixed it. This being
sed-and-makefile stuff I'm sure there are others who are more fluent in
it than me :-)
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2004-10-06 14:31:54 | Re: Win32 VERSIONINFO fix |
Previous Message | Tom Lane | 2004-10-06 14:20:15 | Re: pg_dumpall bug in PG 8.0b3 Win32 port |