Re: [pgsql-hackers-win32] Data directory with trailing [back]slash - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: [pgsql-hackers-win32] Data directory with trailing [back]slash
Date
Msg-id 200407111325.i6BDPKm25502@candle.pha.pa.us
Whole thread Raw
In response to Re: [pgsql-hackers-win32] Data directory with trailing [back]slash  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: [pgsql-hackers-win32] Data directory with trailing [back]slash
List pgsql-patches
I am now confused about the original report.  I don't see how my fix
would correct the reported problem.  trim_trailing_separator() would
have handled d:\pgdata\ and d:\pgdata just fine.  The fix only corrects
d:\.

Magnus, does current CVS fix the problem?

---------------------------------------------------------------------------

Andrew Dunstan wrote:
>
>
> Tom Lane wrote:
>
> >"Magnus Hagander" <mha@sollentuna.net> writes:
> >
> >
> >>It's not possible to start the postmaster on win32 with:
> >>postmaster -D d:\pgdata\
> >>or
> >>postmaster -D d:/pgdata/
> >>
> >>
> >
> >Sounds like canonicalize_path() needs to be applied a bit sooner than
> >it is.
> >
> >BTW I think canonicalize_path() is a few bricks shy of a load yet:
> >I'm not sure it works well with Windows drive-letters, and it definitely
> >will strip significant slashes when given input like '/' or 'C:\'.
> >Feel free to fix those problems while at it...
> >
> >
>
> Or use the attached patch, which I think does it right.
>
> cheers
>
> andrew


>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: the planner will ignore your desire to choose an index scan if your
>       joining column's datatypes do not match

--
  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: Bruce Momjian
Date:
Subject: Re: [pgsql-hackers-win32] Re : Win32 binaries test / pg_dump
Next
From: Bruce Momjian
Date:
Subject: Re: [pgsql-hackers-win32] Data directory with trailing [back]slash