Overview

Request 807113 accepted

- Register with pam-config in %post(un)

Loading...

Dominique Leuenberger's avatar

+Requires(post): coreutils pam-config => coreutils? after we worked so hard in getting rid of that dependency for busybox based containers?


Fabian Vogt's avatar

Should I use /usr/bin/: and /usr/bin/[ instead?


Dominique Leuenberger's avatar

I don't think [ and : need anything in /usr/bin; for [ there is a file in /usr/bin, but it's a shell builtin, so the binary is actually not used by default. And this is even supported by posix shell (/bin/sh is the default shell used by %post scriptlets)


Dominique Leuenberger's avatar
> sh
sh-5.0$ type [
[ is a shell builtin
sh-5.0$ type :
: is a special shell builtin

Dominique Leuenberger's avatar

as alternative to : you could use true (to not use special shell builtins)


Fabian Vogt's avatar

I'll just drop coreutils then

Request History
Fabian Vogt's avatar

favogt created request

- Register with pam-config in %post(un)


GNOME Review Bot's avatar

gnome-review-bot accepted review

Check script succeeded


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar approved review


Dominique Leuenberger's avatar

dimstar accepted request

xin+

openSUSE Build Service is sponsored by