Re: [PATCH] Make "psql -1 < file.sql" work as with "-f" - Mailing list pgsql-hackers

From David Fetter
Subject Re: [PATCH] Make "psql -1 < file.sql" work as with "-f"
Date
Msg-id 20120808225022.GA13591@fetter.org
Whole thread Raw
In response to Re: [PATCH] Make "psql -1 < file.sql" work as with "-f"  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: [PATCH] Make "psql -1 < file.sql" work as with "-f"
List pgsql-hackers
On Wed, Aug 08, 2012 at 04:55:43PM -0400, Robert Haas wrote:
> On Wed, Aug 1, 2012 at 4:28 AM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:
> > Dear PostgreSQL developers,
> >
> > Plese find attached a patch so that:
> >
> >     Make "psql -1 < file.sql" work as with "-f"
> >
> >     Make psql --single-transaction option work on a non-interactive
> >     standard input as well, so that "psql -1 < input.sql" behaves as
> >     "psql -1 -f input.sql".
> >
> > This saner/less error-prone behavior was discussed in this thread back in
> > June:
> >
> >         http://archives.postgresql.org/pgsql-hackers/2012-06/msg00785.php
> >
> > I have tested it manually and it works for me. I'm not sure this is the best
> > possible implementation, but it is a small diff one. I haven't found a place
> > in the regression tests where "psql" could be tested with different options.
> > Did I miss something?
> 
> I'm wondering if perhaps -- in addition to what you've done here -- we
> should make "psql -1" error out if reading from a terminal.

+1 for this.

> Because accepting options that are intended to cause important
> behavior changes and then ignoring those options is Bad.

Yes, It Is.

Cheers,
David.
-- 
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: -Wformat-zero-length
Next
From: Bruce Momjian
Date:
Subject: Re: -Wformat-zero-length