Re: pg_dumpall bug in PG 8.0b3 Win32 port

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "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:06:04
Message-ID: 200410061706.i96H65H27046@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

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.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Andrew Dunstan 2004-10-06 17:15:49 Re: pg_dumpall bug in PG 8.0b3 Win32 port
Previous Message Bruce Momjian 2004-10-06 17:02:08 Re: pg_dumpall bug in PG 8.0b3 Win32 port

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2004-10-06 17:15:49 Re: pg_dumpall bug in PG 8.0b3 Win32 port
Previous Message Bruce Momjian 2004-10-06 17:02:08 Re: pg_dumpall bug in PG 8.0b3 Win32 port