Re: pg_dumpall bug in PG 8.0b3 Win32 port

Andrew Dunstan wrote:
> 
> 
> Bruce Momjian wrote:
> 
> >Tom Lane wrote:
> >  
> >
> >>"Luiz K. Matsumura"  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