> Brent Verner wrote:
> > [2002-01-03 14:19] Bruce Momjian said:
> > |
> > | Actually, we can just do:
> > |
> > | > *)
> > | > dbname=$1
> > | > [ "$#" -ne 1 ] && usage=1
> > |
> > | Meaning if they have anything after the dbname, it is an error. This
> > | catches flags _after_ the dbname. Seems most of the script have this
> > | problem. If people want it fixed, I can easily do it; just give me
> > | go-ahead.
> > +1
> > I can't see a reason to /not/ fix something this simple for the 7.2
> > release. In general, I think it's best to fix things like this
> > "on sight" as opposed to queueing them in TODO where they /might/ sit
> > untouched through another release cycle.
> >  meaning problems that require little effort to fix, and whose
> > solutions are /very/ localized.
> OK, one more +1 and I will get to it.
1: It's not a regression from 7.1. Anything else is too late.
2: The issue does not cause problems if you stick to the documented syntax
does not cause hazard if you don't.
3: The patch is wrong, because showing the usage screen in case of an error
4: Even beginning to talk of "localized", "trivial", "little effort" should
an automatic ban on programming for 1 month.
GMX - Die Kommunikationsplattform im Internet.
Description: application/octet-stream (0 bytes)
In response to
pgsql-hackers by date
|Next:||From: Thomas Lockhart||Date: 2002-01-03 23:03:20|
|Subject: Re: More problem with scripts|
|Previous:||From: Jason Earl||Date: 2002-01-03 22:13:29|
|Subject: Re: SET DATESTYLE to time_t style for client libraries?|