Re: pg_ctl - tighten command parameter checking

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

In response to

Responses

Browse pgsql-hackers by date

  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