SPECS: pygopherd.spec - exits are back '|| exit 1' -> '|| exit $?'...

[Tomasz Wittner]

On Wednesday 01 of September 2004 09:17, Marcin Gajda wrote:
> On Wed, Sep 01, 2004 at 03:29:55AM +0000, twittner wrote:
> > - exits are back '|| exit 1' -> '|| exit $?' (fixed previous change).
>
> To samo jest w gofish.spec. 
Znaczy tyle, że być moze jest tak samo źle.
> Nie kumam tego $?
Nie dziwię się: $? mają dosyć zakręcone kształty.
man ksh:
?      The exit status of the last non-asynchronous  command  executed.
              If  the  last  command  was killed by a signal, $? is set to 128
              plus the signal number.
> , więc nie wiem, czy  
> zmienić.
Ponieważ na lists.pld-linux.org nie mogę znaleźć swojego posta (flame buffer 
overflow in mailman?) w, to go wkleję.
<quote>
Re: [undefine w aramin.net: Fwd: SOURCES: pygopherd-conf.patch (NEW) - root dir 
for PLD]
From: Tomasz Wittner <Tomasz.Wittner w xl.wp.pl>
To: pld-devel-pl w pld-linux.org
Date: Tue Aug 31 22:56:49 2004

Q. odnośnie speca:
/usr/sbin/useradd -u 13 -g 30 -d /no/home -s /bin/false -c "gopherd user" \   
gopher || exit 1

Dlaczego exit 1 ? Dlaczego nie kod powrótu polecenia, które zawiodło?

shadow-4.0.4.1/src/useradd.c:
[...]
/*
 * exit status values
 */
#define E_SUCCESS       0       /* success */
#define E_PW_UPDATE     1       /* can't update password file */
#define E_USAGE         2       /* invalid command syntax */
#define E_BAD_ARG       3       /* invalid argument to option */
#define E_UID_IN_USE    4       /* uid already in use (and no -o) */
#define E_NOTFOUND      6       /* specified group doesn't exist */
#define E_NAME_IN_USE   9       /* username already in use */
#define E_GRP_UPDATE    10      /* can't update group file */
#define E_HOMEDIR       12      /* can't create home directory */
[...]

Jeżeli nie będzie się zastępowało tego exit 1 to przynajmiej będzie wiadomo, 
co się stało - a tak to kłamie.
</quote>
Nikt nie protestował, nikt nic nie miał do powiedzenia, to zmieniłem tak jak 
uważałem - jak jest dobrze - zostawić i przerobić "na tak samo" w innych 
specach - jak jest to źle - wywalić i wytłumaczyć mi why exit 1 jest lepsze.
>
> Pozdrawiam,
Ja również.
-- 
Tomasz Wittner