From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Luiz K(dot) Matsumura" <luiz(at)planit(dot)com(dot)br>, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: pg_dumpall bug in PG 8.0b3 Win32 port |
Date: | 2004-10-06 17:15:49 |
Message-ID: | 41642845.9010009@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-patches |
Bruce Momjian wrote:
>Tom Lane wrote:
>
>
>>"Luiz K. Matsumura" <luiz(at)planit(dot)com(dot)br> writes:
>>
>>
>>>I modified pg_dumpall.c lines between 160 and 270 to add a #ifndef
>>>WIN32... #endif clause to pgdumpopts to store
>>>the parameters with double quotations intead simple quotation marks in
>>>case of WIN32 ports
>>>
>>>
>>This seems unnecessarily ugly. Why not just go over to double quotes?
>>
>>
>
>The patch was actually posted backward so maybe it wasn't clear but the
>change is to use double quotes instead of single quotes for Win32
>arguments just like we do in other cases in pg_dumpall.c.
>
>Single quotes do not behave the same as single quotes in the WIn32
>CMD.EXES processor. For example, DIR "" and DIR '' produce different
>results.
>
>
Yes, but wasn't Tom's point that we could use double quotes on Unix too?
That makes sense to me unless we expect some horrible interpolation effects.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2004-10-06 17:23:29 | Re: pg_dumpall bug in PG 8.0b3 Win32 port |
Previous Message | Bruce Momjian | 2004-10-06 17:06:04 | Re: pg_dumpall bug in PG 8.0b3 Win32 port |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2004-10-06 17:21:55 | Re: pg_config fixes |
Previous Message | Bruce Momjian | 2004-10-06 17:06:04 | Re: pg_dumpall bug in PG 8.0b3 Win32 port |