]> code.ossystems Code Review - openembedded-core.git/commit
util-linux: fix "su -" and package su separately
authorPatrick Ohly <patrick.ohly@intel.com>
Wed, 12 Jul 2017 08:44:11 +0000 (10:44 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 17 Jul 2017 12:49:02 +0000 (13:49 +0100)
commit1ad32536c2d4a3e55e698d27d827caee788cb0f7
treeaea1d154cb9d4af64ddaa3089332402195374642
parent7d3e734481e5d400d03ffd0a12669913fd264c5f
util-linux: fix "su -" and package su separately

"su -" == "su --login" was broken because it uses /etc/pam.d/su-l and
lacking that, falls back to /etc/pam.d/other which denies the
operation. The fix is to symlink "su-l" to the normal "su" pam config
file.

Because "su" usually comes from "shadow" and has been broken like this
without anyone noticing, it probably is not used much and thus should
be packaged separately so that it can be installed only when really
needed. For backwards compatibility, "util-linux" still pulls it in.

It is a bit strange that DISTRO_FEATURES are getting checked when
deciding whether the packages should be defined. It is not wrong, the
packages will be simply empty and thus probably not created when the
distro feature is on and the package config is off. Perhaps there is a
reason, so this is kept unchanged. The symlink however only gets
created when su.util-linux really gets built.

[YOCTO #11126]

Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-core/util-linux/util-linux.inc