diff options
author | Khem Raj <raj.khem@gmail.com> | 2019-08-14 22:56:54 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-08-21 15:29:01 +0100 |
commit | 634b65b7ffa37f499d507ff3d6ba2e49254ae05b (patch) | |
tree | 596dd02763224a8108e450010a7680944b3159ef /meta/recipes-core | |
parent | a3cfd5c366fc261240893e3a4ae8d9230773a12e (diff) | |
download | poky-634b65b7ffa37f499d507ff3d6ba2e49254ae05b.tar.gz |
util-linux: Make pam specific logic apply to target recipe alone
This helps with a case where a distro builds one image with systemd and
another with sysvinit, it ends up recompiling almost everything since
python3-native gets rebuilt and tracing dependencies with
bitbake-diffsigs shows that the chain ends at util-linux-native being
recompiled because distro features now does or does not have 'pam'
Hash for dependent task python/python3_3.7.4.bb:do_prepare_recipe_sysroot:virtual:native:/mnt/a/yoe/sources/openembedded-core/meta/recipes-devt
ools/python/python3_3.7.4.bb changed from 8befaac4f995aaff3f95d27c9caaf1006f86e1344b02c1ae82f5d12f885f2240 to 2a45fe0cd0d3640a88c4a5c8b1880c4e9
a089cc7446a91d2a920c1cef6fa916a
Hash for dependent task util-linux/util-linux_2.34.bb:do_populate_sysroot:virtual:native:/mnt/a/yoe/sources/openembedded-core/meta/recipes-
core/util-linux/util-linux_2.34.bb changed from 0db292cb2e37d5788bdcf51038b2802d748b719d860aca3a26d7a793b0cf3905 to 15d6e165f025f10c2c455df8a87
5cafe021eaed4214c793e708d4827a58ca89d
Hash for dependent task util-linux/util-linux_2.34.bb:do_install:virtual:native:/mnt/a/yoe/sources/openembedded-core/meta/recipes-core/util-linux/util-linux_2.34.bb changed from 54bb4ee6bdb5c7fc260dabddb4932cb0e554a62cd92aba080a18306291fb470b to e25b1119ce8dd7ca43fbd2db771e04fa
6ff6b9d701fd78ac6c443224b036ed9f basehash changed from 8e8687a866689a697001dedc0a43f478e68e6efe270bd77362f24c6000f9e882 to 62df6610eab9c1b1a17d7132943507641c8538690
f26186843c86144d4598e64
Variable do_install value changed:
rm -f ${D}${bindir}/chkdupexe
- if [ "${@bb.utils.filter('DISTRO_FEATURES', 'pam', d)}" ]; then
+ if [ "${@bb.utils.filter('PACKAGECONFIG', 'pam', d)}" ]; then
install -d ${D}${sysconfdir}/pam.d
install -m 0644 ${WORKDIR}/runuser.pamd ${D}${sysconfdir}/pam.d/runuser
install -m 0644 ${WORKDIR}/runuser-l.pamd ${D}${sysconfdir}/pam.d/runuser-l
@@ -47,5 +47,4 @@
rm -f ${D}${base_sbindir}/nologin
rm -f ${D}${base_bindir}/kill
-DISTRO_FEATURES{pam} = Unset
PACKAGECONFIG{pam} = Unset
So far it seems this pam conditional code in util-linux is target
specific and would not apply to native or nativesdk recipes
(From OE-Core rev: a57f56576eda4b3ad5496e375b432e06e34ad8b9)
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-core')
-rw-r--r-- | meta/recipes-core/util-linux/util-linux.inc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/meta/recipes-core/util-linux/util-linux.inc b/meta/recipes-core/util-linux/util-linux.inc index 84c7012752..1fa82363b1 100644 --- a/meta/recipes-core/util-linux/util-linux.inc +++ b/meta/recipes-core/util-linux/util-linux.inc | |||
@@ -190,20 +190,19 @@ do_install () { | |||
190 | echo 'MOUNTALL="-t nonfs,nosmbfs,noncpfs"' > ${D}${sysconfdir}/default/mountall | 190 | echo 'MOUNTALL="-t nonfs,nosmbfs,noncpfs"' > ${D}${sysconfdir}/default/mountall |
191 | 191 | ||
192 | rm -f ${D}${bindir}/chkdupexe | 192 | rm -f ${D}${bindir}/chkdupexe |
193 | } | ||
193 | 194 | ||
194 | if [ "${@bb.utils.filter('DISTRO_FEATURES', 'pam', d)}" ]; then | 195 | do_install_append_class-target () { |
196 | if [ "${@bb.utils.filter('PACKAGECONFIG', 'pam', d)}" ]; then | ||
195 | install -d ${D}${sysconfdir}/pam.d | 197 | install -d ${D}${sysconfdir}/pam.d |
196 | install -m 0644 ${WORKDIR}/runuser.pamd ${D}${sysconfdir}/pam.d/runuser | 198 | install -m 0644 ${WORKDIR}/runuser.pamd ${D}${sysconfdir}/pam.d/runuser |
197 | install -m 0644 ${WORKDIR}/runuser-l.pamd ${D}${sysconfdir}/pam.d/runuser-l | 199 | install -m 0644 ${WORKDIR}/runuser-l.pamd ${D}${sysconfdir}/pam.d/runuser-l |
198 | fi | ||
199 | if [ "${@bb.utils.filter('PACKAGECONFIG', 'pam', d)}" ]; then | ||
200 | # Required for "su -" aka "su --login" because | 200 | # Required for "su -" aka "su --login" because |
201 | # otherwise it uses "other", which has "auth pam_deny.so" | 201 | # otherwise it uses "other", which has "auth pam_deny.so" |
202 | # and thus prevents the operation. | 202 | # and thus prevents the operation. |
203 | ln -s su ${D}${sysconfdir}/pam.d/su-l | 203 | ln -s su ${D}${sysconfdir}/pam.d/su-l |
204 | fi | 204 | fi |
205 | } | 205 | } |
206 | |||
207 | # nologin causes a conflict with shadow-native | 206 | # nologin causes a conflict with shadow-native |
208 | # kill causes a conflict with coreutils-native (if ${bindir}==${base_bindir}) | 207 | # kill causes a conflict with coreutils-native (if ${bindir}==${base_bindir}) |
209 | do_install_append_class-native () { | 208 | do_install_append_class-native () { |