Re: psql: small patch to correct filename formatting error in '\s FILE' output - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: psql: small patch to correct filename formatting error in '\s FILE' output
Date
Msg-id 20130910012500.GE32173@momjian.us
Whole thread Raw
In response to Re: psql: small patch to correct filename formatting error in '\s FILE' output  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: psql: small patch to correct filename formatting error in '\s FILE' output
List pgsql-hackers
On Tue, Jan 22, 2013 at 07:30:59PM -0500, Tom Lane wrote:
> Ian Lawrence Barwick <barwick@gmail.com> writes:
> > Related email from the archives on this subject:
> > http://www.postgresql.org/message-id/37ed240d0611200645l5b70c8ddw5fb735e0d35a7b22@mail.gmail.com
> 
> I agree with the opinion stated there that \cd with no argument really
> ought to do what "cd" with no argument usually does on the platform.
> So if we're going to fix \cd to print the resulting current directory,
> wouldn't it work to just set "dir" to "." rather than "/" for Windows?
> 
> > Does commit 0725065b just need to be reverted, or is an additional
> > patch required to remove the prefixed working directory from \s output?
> 
> Offhand it looked like reverting the commit would be enough, but I
> didn't look hard to see if there had been any subsequent related
> changes.  [ pokes around... ]  Well, at least there are still no other
> uses of pset.dirname.

I still see that weird behavior in git head:
 pgdevel=# \s history.txt Wrote history to file "./history.txt". pgdevel=# \s /tmp/history.txt Wrote history to file
".//tmp/history.txt".pgdevel=# \cd /tmp pgdevel=# \s /tmp/history.txt Wrote history to file "/tmp//tmp/history.txt".
 

Should I revert the suggested patch?

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: New statistics for WAL buffer dirty writes
Next
From: Peter Geoghegan
Date:
Subject: Re: New statistics for WAL buffer dirty writes