[OE-core] [PATCH v6] rootfs: Modify RPM installation
Burton, Ross
ross.burton at intel.com
Thu Dec 22 14:09:32 UTC 2016
Small problem: we always build rpm-native as we need it for some tools, but
rpmlibdir is only defined in package_rpm.bbclass which doesn't have to be
inherited.
This leads to just --with-path-database= being passed to configure. What's
the behaviour of RPM when this happens? What does it default to? If it's
going to be safe (ie a sane default is used), please update the commit
message to explain this.
Ross
On 22 December 2016 at 10:23, David Vincent <freesilicon at gmail.com> wrote:
> When using a custom RPM data directory instead of the default dir
> '/var/lib/rpm', the final image did not contain any of the database
> files in the expected location. This commit takes into account the
> 'rpmlibdir' variable set into 'rootfs_rpm.bbclass'.
>
> Changes since v4:
> - Remove boolean in d.getVar() calls.
>
> Changes since v5:
> - Also modify rpm recipe to build the database in the correct
> location and avoid errors during rootfs creation.
>
> Signed-off-by: David Vincent <freesilicon at gmail.com>
> ---
> meta/lib/oe/package_manager.py | 23 +++++++++++++----------
> meta/lib/oe/rootfs.py | 3 ++-
> meta/recipes-devtools/rpm/rpm_5.4.16.bb | 2 ++
> 3 files changed, 17 insertions(+), 11 deletions(-)
>
> diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.
> py
> index e557473999..3f2878c394 100644
> --- a/meta/lib/oe/package_manager.py
> +++ b/meta/lib/oe/package_manager.py
> @@ -356,7 +356,8 @@ class RpmPkgsList(PkgsList):
> super(RpmPkgsList, self).__init__(d, rootfs_dir)
>
> self.rpm_cmd = bb.utils.which(os.getenv('PATH'), "rpm")
> - self.image_rpmlib = os.path.join(self.rootfs_dir, 'var/lib/rpm')
> + self.rpm_libdir = self.d.getVar('rpmlibdir')
> + self.image_rpmlib = os.path.join(self.rootfs_dir,
> self.rpm_libdir[1:])
>
> self.ml_prefix_list, self.ml_os_list = \
> RpmIndexer(d, rootfs_dir).get_ml_prefix_and_os_list(arch_var,
> os_var)
> @@ -416,7 +417,7 @@ class RpmPkgsList(PkgsList):
>
> def list_pkgs(self):
> cmd = [self.rpm_cmd, '--root', self.rootfs_dir]
> - cmd.extend(['-D', '_dbpath /var/lib/rpm'])
> + cmd.extend(['-D', '_dbpath', self.rpm_libdir])
> cmd.extend(['-qa', '--qf', '[%{NAME} %{ARCH} %{VERSION}
> %{PACKAGEORIGIN}\n]'])
>
> try:
> @@ -683,7 +684,8 @@ class RpmPM(PackageManager):
> self.solution_manifest = self.d.expand('${T}/saved/%s_solution' %
> self.task_name)
> self.saved_rpmlib = self.d.expand('${T}/saved/%s' %
> self.task_name)
> - self.image_rpmlib = os.path.join(self.target_rootfs,
> 'var/lib/rpm')
> + self.rpm_libdir = self.d.getVar('rpmlibdir')
> + self.image_rpmlib = os.path.join(self.target_rootfs,
> self.rpm_libdir[1:])
>
> if not os.path.exists(self.d.expand('${T}/saved')):
> bb.utils.mkdirhier(self.d.expand('${T}/saved'))
> @@ -954,7 +956,7 @@ class RpmPM(PackageManager):
> open(db_config_dir, 'w+').write(DB_CONFIG_CONTENT)
>
> # Create database so that smart doesn't complain (lazy init)
> - cmd = [self.rpm_cmd, '--root', self.target_rootfs, '--dbpath',
> '/var/lib/rpm', '-qa']
> + cmd = [self.rpm_cmd, '--root', self.target_rootfs, '--dbpath',
> self.rpm_libdir, '-qa']
> try:
> subprocess.check_output(cmd, stderr=subprocess.STDOUT)
> except subprocess.CalledProcessError as e:
> @@ -963,20 +965,19 @@ class RpmPM(PackageManager):
> # Import GPG key to RPM database of the target system
> if self.d.getVar('RPM_SIGN_PACKAGES') == '1':
> pubkey_path = self.d.getVar('RPM_GPG_PUBKEY')
> - cmd = [self.rpm_cmd, '--root', self.target_rootfs,
> '--dbpath', '/var/lib/rpm', '--import', pubkey_path]
> + cmd = [self.rpm_cmd, '--root', self.target_rootfs,
> '--dbpath', self.rpm_libdir, '--import', pubkey_path]
> try:
> subprocess.check_output(cmd, stderr=subprocess.STDOUT)
> except subprocess.CalledProcessError as e:
> bb.fatal("Import GPG key failed. Command '%s' "
> "returned %d:\n%s" % (' '.join(cmd),
> e.returncode, e.output.decode("utf-8")))
>
> -
> # Configure smart
> bb.note("configuring Smart settings")
> bb.utils.remove(os.path.join(self.target_rootfs,
> 'var/lib/smart'),
> True)
> self._invoke_smart(['config', '--set', 'rpm-root=%s' %
> self.target_rootfs])
> - self._invoke_smart(['config', '--set', 'rpm-dbpath=/var/lib/rpm'])
> + self._invoke_smart(['config', '--set', 'rpm-dbpath=%s' %
> self.rpm_libdir])
> self._invoke_smart(['config', '--set', 'rpm-extra-macros._var=%s'
> %
> self.d.getVar('localstatedir')])
> cmd = ["config", "--set", "rpm-extra-macros._tmppath=/%s/tmp" %
> self.install_dir_name]
> @@ -1236,7 +1237,7 @@ class RpmPM(PackageManager):
>
> if not with_dependencies:
> cmd = [self.rpm_cmd] + ["-e", "--nodeps", "--root=%s" %
> - self.target_rootfs, "--dbpath=/var/lib/rpm",
> + self.target_rootfs, "--dbpath=%s" % self.rpm_libdir,
> "--define='_cross_scriptlet_wrapper %s'" %
> self.scriptlet_wrapper,
> "--define='_tmppath /%s/tmp'" %
> self.install_dir_name] + pkgs
> @@ -1384,7 +1385,8 @@ class RpmPM(PackageManager):
> saved_dir = self.target_rootfs + self.d.expand('${sysconfdir}/rpm-postinsts/')
> + new_pkg
>
> cmd = self.rpm_cmd + ' -q --scripts --root ' + self.target_rootfs
> - cmd += ' --dbpath=/var/lib/rpm ' + new_pkg
> + cmd += ' --dbpath=%s ' % self.rpm_libdir
> + cmd += new_pkg
> cmd += ' | sed -n -e "/^postinstall scriptlet (using .*):$/,/^.*
> scriptlet (using .*):$/ {/.*/p}"'
> cmd += ' | sed -e "/postinstall scriptlet (using \(.*\)):$/d"'
> cmd += ' -e "/^.* scriptlet (using .*):$/d" > %s' % saved_dir
> @@ -1416,7 +1418,8 @@ class RpmPM(PackageManager):
> '''
> def unlock_rpm_db(self):
> # Remove rpm db lock files
> - rpm_db_locks = glob.glob('%s/var/lib/rpm/__db.*' %
> self.target_rootfs)
> + rpm_db_locks = glob.glob('%s%s/__db.*' % (self.target_rootfs,
> + self.rpm_libdir))
> for f in rpm_db_locks:
> bb.utils.remove(f, True)
>
> diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
> index d9a473006a..6eb05feb6c 100644
> --- a/meta/lib/oe/rootfs.py
> +++ b/meta/lib/oe/rootfs.py
> @@ -429,6 +429,7 @@ class RpmRootfs(Rootfs):
>
> def _create(self):
> pkgs_to_install = self.manifest.parse_initial_manifest()
> + rpm_libdir = self.d.getVar('rpmlibdir')
> rpm_pre_process_cmds = self.d.getVar('RPM_PREPROCESS_COMMANDS')
> rpm_post_process_cmds = self.d.getVar('RPM_POSTPROCESS_COMMANDS')
>
> @@ -476,7 +477,7 @@ class RpmRootfs(Rootfs):
> if self.progress_reporter:
> self.progress_reporter.next_stage()
>
> - self._setup_dbg_rootfs(['/etc/rpm', '/var/lib/rpm',
> '/var/lib/smart'])
> + self._setup_dbg_rootfs(['/etc/rpm', rpm_libdir,
> '/var/lib/smart'])
>
> execute_pre_post_process(self.d, rpm_post_process_cmds)
>
> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.16.bb
> b/meta/recipes-devtools/rpm/rpm_5.4.16.bb
> index 3df4d1ff1d..12181d89b1 100644
> --- a/meta/recipes-devtools/rpm/rpm_5.4.16.bb
> +++ b/meta/recipes-devtools/rpm/rpm_5.4.16.bb
> @@ -342,6 +342,7 @@ EXTRA_OECONF += "--verbose \
> --enable-build-warnings \
> --enable-build-debug \
> --enable-maintainer-mode \
> + --with-path-database=${rpmlibdir} \
> --with-path-macros=${rpm_macros} \
> --with-path-lib=${libdir}/rpm \
> --with-bugreport=http://bugzilla.yoctoproject.org \
> @@ -384,6 +385,7 @@ FILES_${PN} = "${bindir}/rpm \
> ${localstatedir}/lib/wdj \
> ${bindir}/rpm.real \
> ${bindir}/rpmconstant.real \
> + ${rpmlibdir} \
> "
>
> FILES_${PN}-common = "${bindir}/rpm2cpio \
> --
> 2.11.0
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20161222/13c05452/attachment-0002.html>
More information about the Openembedded-core
mailing list