From: Trevor Woerner <[email protected]>

The cups' PACKAGECONFIG is populated based on DISTRO_FEATURES, but a user
is free to enable or disable PACKAGECONFIGs at will. In theory it is
possible that pam is enabled globally in DISTRO_FEATURES but disabled in
cups' PACKAGECONFIG. Checking the PACKAGECONFIG to determine whether or not
pam is enabled would be a safer check rather than relying on DISTRO_FEATURES.

Signed-off-by: Trevor Woerner <[email protected]>
Signed-off-by: Alexandre Belloni <[email protected]>
(cherry picked from commit a053dd177ddc99ced11e68914079be0ffe261262)
Signed-off-by: Steve Sakoman <[email protected]>
---
 meta/recipes-extended/cups/cups.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/cups/cups.inc 
b/meta/recipes-extended/cups/cups.inc
index b8b655a4fb..1b522d7d85 100644
--- a/meta/recipes-extended/cups/cups.inc
+++ b/meta/recipes-extended/cups/cups.inc
@@ -74,7 +74,7 @@ do_install () {
        rmdir ${D}/${libexecdir}/${BPN}/driver
 
        # Fix the pam configuration file permissions
-       if ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'true', 'false', d)}; 
then
+       if ${@bb.utils.contains('PACKAGECONFIG', 'pam', 'true', 'false', d)}; 
then
            chmod 0644 ${D}${sysconfdir}/pam.d/cups
        fi
 
-- 
2.34.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178726): 
https://lists.openembedded.org/g/openembedded-core/message/178726
Mute This Topic: https://lists.openembedded.org/mt/97660869/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to