[2002-01-04 00:34] Bruce Momjian said:
| Brent Verner wrote:
| > [2002-01-03 23:33] Peter Eisentraut said:
| > | > Brent Verner wrote:
| > | > > [2002-01-03 14:19] Bruce Momjian said:
| > | > > |
| > | > > | Actually, we can just do:
| >
| > | > > +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[1]
| > | > > "on sight" as opposed to queueing them in TODO where they /might/ sit
| > | > > untouched through another release cycle.
| > | > >
| > | > > [1] meaning problems that require little effort to fix, and whose
| > | > > solutions are /very/ localized.
| > | >
| > | > OK, one more +1 and I will get to it.
| > |
| > | -4
|
| OK, here's the patch. Seems createdb wasn't properly handling the db
| comment (arg code now similar to createlang), createlang dbname being
| optional wasn't documented in --help, and vacuumdb wasn't handlling an
| optional dbname. I added the required checks so extra arguments report
| a failure:
|
| $ dropdb lkjas asdfljk test
| dropdb: invalid option: asdfljk
| Try 'dropdb --help' for more information.
This looks good to me.
thanks. brent
--
"Develop your talent, man, and leave the world something. Records are
really gifts from people. To think that an artist would love you enough
to share his music with anyone is a beautiful thing." -- Duane Allman