From: Saul Wold <sgw@linux.intel.com>
To: Laurentiu Palcu <laurentiu.palcu@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/7] run-postinsts refactoring
Date: Thu, 23 May 2013 20:38:25 -0700 [thread overview]
Message-ID: <519EE0B1.6090702@linux.intel.com> (raw)
In-Reply-To: <cover.1369313581.git.laurentiu.palcu@intel.com>
On 05/23/2013 06:15 AM, Laurentiu Palcu wrote:
> Hi all,
>
> My work at #4484 revealed that the package managers deb/ipk/rpm handle removal
> of PM meta-data (when the PM is not deployed) and the delayed postinstalls
> execution in their own way. Currently we have:
> * run-postinsts (for deb/ipk) and rpm-postinsts (for rpm) for running the delayed
> postinstalls on target when the PM is not part of the image;
> * opkg removes the meta-data and some uneeded packages whilst deb/rpm remove
> only the meta-data;
> * both opkg and deb have no way of running the delayed postinstalls on target,
> if package-management is not part of IMAGE_FEATURES, because the meta-data
> was deleted (and it would have been useless anyway since the PM is not installed);
>
> That being said, this patchset tries to create a more unified solution for all
> PM backends.
>
> rpm-postinsts recipe is replaced by run-postinsts but I'll keep it in oe-core
> for a while (a couple of weeks maybe), just in case the current solution does not
> satisfy the RPM users (I doubt it though).
>
> I did tests with core-image-minimal using sysvinit/systemd and with/without delayed
> postinstalls. The results are as expected.
>
This seems to have a problem building on the auto builder, most of the
failures are due to the following:
/usr/sbin/update-rc.d: /bin/sh: bad interpreter: No such file or directory
Details here:
> | log_check: Using /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-ppc/build/build/tmp/work/qemuppc-poky-linux/core-image-minimal-dev/1.0-r0/temp/log.do_rootfs.32640 as logfile
> | Logfile is clean
> | rpm -e --nodeps --root=/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-ppc/build/build/tmp/work/qemuppc-poky-linux/core-image-minimal-dev/1.0-r0/rootfs --dbpath=/var/lib/rpm update-rc.d base-passwd run-postinsts
> | /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-ppc/build/build/tmp/sysroots/x86_64-linux/var/tmp/rpm-tmp.67450: /usr/sbin/update-rc.d: /bin/sh: bad interpreter: No such file or directory
> | error: %postun(run-postinsts-1.0-r9.ppc603e) scriptlet failed, exit status 126
> | ERROR: Function failed: do_rootfs (see /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-ppc/build/build/tmp/work/qemuppc-poky-linux/core-image-minimal-dev/1.0-r0/temp/log.do_rootfs.32640 for further information)
> NOTE: recipe core-image-minimal-dev-1.0-r0: task do_rootfs: Failed
> ERROR: Task 71 (/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-ppc/bu
Sau!
> Thanks,
> Laurentiu
>
> The following changes since commit f7b639e527a65f9ce77aef07651ac5ec1e094e52:
>
> rpm-postinsts.bb: enable postinst logging (2013-05-23 10:28:05 +0100)
>
> are available in the git repository at:
>
> git://git.yoctoproject.org/poky-contrib lpalcu/run_postinsts_refactor
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=lpalcu/run_postinsts_refactor
>
> Laurentiu Palcu (7):
> systemd: add udev-utils dependency for udev-hwdb
> rootfs_*.bbclass: add some helper functions
> image.bbclass: remove unneeded files from the image
> core-image-minimal: do not remove packaging files in the rootfs
> postprocess
> run-postinsts: make it generic
> rootfs_rpm.bbclas: switch to using run-postinsts
> rpm: replace rpm-postinsts dependency with run-postinsts
>
> meta/classes/image.bbclass | 33 +++++++++++
> meta/classes/rootfs_deb.bbclass | 23 +++++++-
> meta/classes/rootfs_ipk.bbclass | 39 ++++++-------
> meta/classes/rootfs_rpm.bbclass | 25 ++++++++-
> meta/recipes-core/images/core-image-minimal.bb | 2 -
> meta/recipes-core/systemd/systemd_199.bb | 2 +
> .../dpkg/run-postinsts/run-postinsts | 59 ++++++++++++--------
> .../dpkg/run-postinsts/run-postinsts.awk | 30 ----------
> meta/recipes-devtools/dpkg/run-postinsts_1.0.bb | 7 +--
> meta/recipes-devtools/rpm/rpm_5.4.9.bb | 2 +-
> 10 files changed, 135 insertions(+), 87 deletions(-)
> delete mode 100644 meta/recipes-devtools/dpkg/run-postinsts/run-postinsts.awk
>
next prev parent reply other threads:[~2013-05-24 3:38 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-23 13:15 [PATCH 0/7] run-postinsts refactoring Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 1/7] systemd: add udev-utils dependency for udev-hwdb Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 2/7] rootfs_*.bbclass: add some helper functions Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 3/7] image.bbclass: remove unneeded files from the image Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 4/7] core-image-minimal: do not remove packaging files in the rootfs postprocess Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 5/7] run-postinsts: make it generic Laurentiu Palcu
2013-05-23 15:19 ` Burton, Ross
2013-05-23 13:15 ` [PATCH 6/7] rootfs_rpm.bbclas: switch to using run-postinsts Laurentiu Palcu
2013-05-23 13:15 ` [PATCH 7/7] rpm: replace rpm-postinsts dependency with run-postinsts Laurentiu Palcu
2013-05-24 3:38 ` Saul Wold [this message]
2013-05-24 10:35 ` [PATCH 0/7] run-postinsts refactoring Laurentiu Palcu
2013-06-06 7:41 ` [PATCH v2 " Laurentiu Palcu
2013-06-06 16:43 ` Mark Hatle
2013-06-06 18:09 ` Laurentiu Palcu
2013-06-06 18:22 ` Mark Hatle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=519EE0B1.6090702@linux.intel.com \
--to=sgw@linux.intel.com \
--cc=laurentiu.palcu@intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.