Re: pg_dumpall bug in PG 8.0b3 Win32 port

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

In response to

Responses

Browse pgsql-bugs by date

  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

Browse pgsql-patches by date

  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