From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Oliver Elphick <olly(at)lfix(dot)co(dot)uk> |
Cc: | <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_ctl - tighten command parameter checking |
Date: | 2002-02-18 16:35:36 |
Message-ID: | Pine.LNX.4.30.0202181133340.684-100000@peter.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Oliver Elphick writes:
> The attached patch improves the command parameter checking of pg_ctl.
>
> At present, there is nothing to check that the parameter given with a
> parameter-taking option is actually valid. For example, -l can be given
> without a following logfile name; on a strict POSIX shell such as ash,
> you will get a subsequent failure because of too many shifts, but bash
> will let it pass without showing any error. The patch checks that each
> parameter is not empty and is not another option.
Isn't this problem present in all of our scripts?
Btw., you shouldn't use "cut" in portable scripts. You could probably use
"case" to do the matching you want.
--
Peter Eisentraut peter_e(at)gmx(dot)net
From | Date | Subject | |
---|---|---|---|
Next Message | Trond Eivind =?iso-8859-1?q?Glomsr=F8d?= | 2002-02-18 16:43:55 | Re: [Fwd: Re: Errors with building PG 7.2 on Mandrake 8.0] |
Previous Message | Bruce Momjian | 2002-02-18 16:07:47 | Logical vs. Physical attribute numbering |