Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: pkgsrc



Hubert Feyrer wrote about Re: CVS commit: pkgsrc:
:On 9 Mar 1999, Geoff Wing wrote:
:> :So dumb-down the shell commands used: make them into
:> :"command; if [ $? -ne 0 ]; then..."
:> :Suggested by David Brownlee.
:> ``if command; then : ; else .....; fi''
:> might be nicer on the eye.
:I don't thinks so. You're doing two things here (1. run the command, 2.
:check if it went ok), and seperating this make things easier to read
:instead of stuffing it all in one command (with one part even unused).

Ah well, just my personal preference.  And it doesn't fork off just to
run ``['' which is not handled internally by most (if any) Bourne shells.

Regards,
-- 
Geoff Wing   <gcw%pobox.com@localhost>            Mobile : (Australia) 0412 162 
441
Work URL: http://www.primenet.com.au/   Ego URL: http://pobox.com/~gcw/



Home | Main Index | Thread Index | Old Index