@Conan-Kudo commented on this pull request.


> +
+hasgroup() {
+       grep "^${1}:" "$ROOT"/etc/group >/dev/null
+}
+
+user() {
+       local user="$1"
+       local uid="$2"
+       local desc="$3"
+       local group="$4"
+       local home="$5"
+       local shell="$6"
+
+       [ "$desc" = '-' ] && desc=
+       { [ "$home" = '-' ] || [ "$home" = '' ]; } && home=/
+       { [ "$shell" = '-' ] || [ "$shell" = '' ]; } && shell=/usr/sbin/nologin

This needs to be `/sbin/login`. We cannot assume it's in `/usr`.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2573#pullrequestreview-1547737657
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2573/review/1547737...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to