Hi Cyril,

Cyril Brulebois wrote:
> (I guess you meant non-invasive in Subject?)

Ehm, yes.

> (For lazy people following at home, it's about debian/init) The patched
> init script would have:
> | set -e
> | …
> | [ -x /sbin/restorecon ] && /sbin/restorecon $SCREENDIR
> 
> Meaning you never get to do the clean-up and the chmod dance if the
> policycoreutils package isn't installed.

Meh. Thanks for spotting this!

if [ -x /sbin/restorecon ]; then
        /sbin/restorecon $SCREENDIR
fi

should be better for "set -e", shouldn't it?

                Regards, Axel
-- 
 ,''`.  |  Axel Beckert <[email protected]>, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
  `-    |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to