summaryrefslogtreecommitdiffstats
path: root/meta
diff options
context:
space:
mode:
authorMing Liu <ming.liu@windriver.com>2014-03-28 17:43:08 +0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2014-04-01 23:37:02 +0100
commita684808899899e88a869639ebda114a86c7358cb (patch)
treea95a670b8aca604a0ddbd00ec6b00ded532ae950 /meta
parent8b63ff3586ddeec7dc1fda271a98d6af019664c0 (diff)
downloadpoky-a684808899899e88a869639ebda114a86c7358cb.tar.gz
libpam: fix multilib packaging issue for pam-plugins
libpam might miss ABI specific dependencies for pam-plugins-*, for RPM uses generic names to check the packages depending on it and doesn't consider the arch, which will lead to packaging issues in mulbilib build. pam_plugin_hook is added because the plugin packages are dynamically generated, so we need to manually process multilib names by add baselib to RPROVIDES/RDEPENDS as ABI specific tag. (From OE-Core rev: d08e64a98316d7659b0fb56812667c534f66a1a8) Signed-off-by: Ming Liu <ming.liu@windriver.com> I worked with Ming Liu on this particular issue. You may wonder why this is necessary let me attempt to explain the underlying causes. In deb/ipk on a multilib package, the package name has specific multilib references in it. I.e. the alternative libraries start with something like lib32-... This was done primarily because deb/ipk do not allow two packages with the same name (but different architectures) to be installed at the same time. So the name has to be unique. In RPM however, the names of the packages and matches with the architectures and if they are not the same we can do these multilib installs. This matches the behavior of other RPM based distributions and in many ways the tools people are used to working with RPM. For the most part this works fine in multilib configurations because additional per-file dependencies are added that capture the shared library dependencies with ABI specific information. This unfortunately fails in a few cases where plugins are dynamically loaded via dlopen -- such as libpam. One possible fix is simply to follow the deb/ipk package naming, but this causes a design advantage of rpm. When a package has a dependency on 'bash', we really don't care what bash is installed, only that -a- bash is installed. In the deb/ipk case, the lib32- packages would end up with a lib32-bash dependency and you could potentially end up with two 'bash' packages being installed. So the fix I recommended for the issue was to add the baselib path to the internal dependencies. Since we know that the libpam installed in 'lib' needs the modules that were compiled to also work with the 'lib' version of libpam. While the libpam in 'lib64' need the modules to work with the 'lib64' version of the plugins. Existing dependencies are preserved so there is no impact in the ipk/deb case, the RPM case is resolved as the additional dependency information is now present for the package manager to select the package we really want. If anyone else has a suggestion for an alternative fix, we're interested -- but this is the best answer we could come up with. (If any of the above should be added to the commit message, the YP bug, or documentation, please let me know and I'll make sure it gets added.) Signed-off-by: Mark Hatle <mark.hatle@windriver.com> [YOCTO #4532] Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r--meta/recipes-extended/pam/libpam_1.1.6.bb29
1 files changed, 25 insertions, 4 deletions
diff --git a/meta/recipes-extended/pam/libpam_1.1.6.bb b/meta/recipes-extended/pam/libpam_1.1.6.bb
index 6ddfcf46b2..61aa0a1ddd 100644
--- a/meta/recipes-extended/pam/libpam_1.1.6.bb
+++ b/meta/recipes-extended/pam/libpam_1.1.6.bb
@@ -62,9 +62,12 @@ FILES_${PN}-xtests = "${datadir}/Linux-PAM/xtests"
62 62
63PACKAGES_DYNAMIC += "^pam-plugin-.*" 63PACKAGES_DYNAMIC += "^pam-plugin-.*"
64 64
65RDEPENDS_${PN}-runtime = "libpam pam-plugin-deny pam-plugin-permit pam-plugin-warn pam-plugin-unix" 65RPROVIDES_${PN} += "libpam-${baselib}"
66RDEPENDS_${PN}-xtests = "libpam pam-plugin-access pam-plugin-debug pam-plugin-cracklib pam-plugin-pwhistory pam-plugin-succeed-if pam-plugin-time coreutils" 66RPROVIDES_${PN}-runtime += "libpam-runtime-${baselib}"
67RRECOMMENDS_${PN} = "libpam-runtime" 67
68RDEPENDS_${PN}-runtime = "libpam-${baselib} pam-plugin-deny-${baselib} pam-plugin-permit-${baselib} pam-plugin-warn-${baselib} pam-plugin-unix-${baselib}"
69RDEPENDS_${PN}-xtests = "libpam-${baselib} pam-plugin-access-${baselib} pam-plugin-debug-${baselib} pam-plugin-cracklib-${baselib} pam-plugin-pwhistory-${baselib} pam-plugin-succeed-if-${baselib} pam-plugin-time-${baselib} coreutils"
70RRECOMMENDS_${PN} = "libpam-runtime-${baselib}"
68 71
69python populate_packages_prepend () { 72python populate_packages_prepend () {
70 def pam_plugin_append_file(pn, dir, file): 73 def pam_plugin_append_file(pn, dir, file):
@@ -74,12 +77,30 @@ python populate_packages_prepend () {
74 nf = of + " " + nf 77 nf = of + " " + nf
75 d.setVar('FILES_' + pn, nf) 78 d.setVar('FILES_' + pn, nf)
76 79
80 def pam_plugin_hook(file, pkg, pattern, format, basename):
81 baselib = d.getVar('baselib', True)
82 mlprefix = d.getVar('MLPREFIX', True) or ''
83
84 rdeps = d.getVar('RDEPENDS_' + pkg, True)
85 if rdeps:
86 rdeps = rdeps + " " + mlprefix + "libpam-" + baselib
87 else:
88 rdeps = mlprefix + "libpam-" + baselib
89 d.setVar('RDEPENDS_' + pkg, rdeps)
90
91 provides = d.getVar('RPROVIDES_' + pkg, True)
92 if provides:
93 provides = provides + " " + pkg + "-" + baselib
94 else:
95 provides = pkg + "-" + baselib
96 d.setVar('RPROVIDES_' + pkg, provides)
97
77 dvar = bb.data.expand('${WORKDIR}/package', d, True) 98 dvar = bb.data.expand('${WORKDIR}/package', d, True)
78 pam_libdir = d.expand('${base_libdir}/security') 99 pam_libdir = d.expand('${base_libdir}/security')
79 pam_sbindir = d.expand('${sbindir}') 100 pam_sbindir = d.expand('${sbindir}')
80 pam_filterdir = d.expand('${base_libdir}/security/pam_filter') 101 pam_filterdir = d.expand('${base_libdir}/security/pam_filter')
81 102
82 do_split_packages(d, pam_libdir, '^pam(.*)\.so$', 'pam-plugin%s', 'PAM plugin for %s', extra_depends='') 103 do_split_packages(d, pam_libdir, '^pam(.*)\.so$', 'pam-plugin%s', 'PAM plugin for %s', hook=pam_plugin_hook, extra_depends='')
83 mlprefix = d.getVar('MLPREFIX', True) or '' 104 mlprefix = d.getVar('MLPREFIX', True) or ''
84 pam_plugin_append_file('%spam-plugin-unix' % mlprefix, pam_sbindir, 'unix_chkpwd') 105 pam_plugin_append_file('%spam-plugin-unix' % mlprefix, pam_sbindir, 'unix_chkpwd')
85 pam_plugin_append_file('%spam-plugin-unix' % mlprefix, pam_sbindir, 'unix_update') 106 pam_plugin_append_file('%spam-plugin-unix' % mlprefix, pam_sbindir, 'unix_update')