Re: pg_dumpall bug in PG 8.0b3 Win32 port - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: pg_dumpall bug in PG 8.0b3 Win32 port
Date
Msg-id 200410061723.i96HNT829498@candle.pha.pa.us
Whole thread Raw
In response to Re: pg_dumpall bug in PG 8.0b3 Win32 port  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-patches
Andrew Dunstan wrote:
>
>
> Bruce Momjian wrote:
>
> >Tom Lane wrote:
> >
> >
> >>"Luiz K. Matsumura" <luiz@planit.com.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.

Oh, OK.  Well, if we use double-quotes we don't allow double-quotes in
the input, as well adding handling of dollar signs and other things.
Single quotes are more robust under Unix and I don't want to risk
changing that.

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

pgsql-patches by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: pg_dumpall bug in PG 8.0b3 Win32 port
Next
From: Bruce Momjian
Date:
Subject: Re: pg_config fixes