> -----Original Message----- > From: Magnus Hagander [mailto:[EMAIL PROTECTED] > Sent: 06 October 2004 15:27 > To: Dave Page; PostgreSQL-patches > Subject: RE: [PATCHES] Win32 VERSIONINFO fix > > > > 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!
Advertising
I never said it was pretty, and as you didn't tell me why Bruce added the quotes when we spoke before... :-) > 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 can do the latter if that's what ppl want. Like you though I'd need some sed help to leave the quotes in the Makefiles without escaping them. /D ---------------------------(end of broadcast)--------------------------- TIP 4: Don't 'kill -9' the postmaster
