From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.windriver.com ([147.11.1.11]) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1UMZrF-00060S-Ks for openembedded-core@lists.openembedded.org; Mon, 01 Apr 2013 10:12:50 +0200 Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail.windriver.com (8.14.5/8.14.3) with ESMTP id r317tdkD024461 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL) for ; Mon, 1 Apr 2013 00:55:39 -0700 (PDT) Received: from [128.224.163.154] (128.224.163.154) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.2.342.3; Mon, 1 Apr 2013 00:55:39 -0700 Message-ID: <51593D83.3020603@windriver.com> Date: Mon, 1 Apr 2013 15:55:47 +0800 From: ChenQi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130106 Thunderbird/17.0.2 MIME-Version: 1.0 To: References: <1364444326-16088-1-git-send-email-zhenhua.luo@freescale.com> In-Reply-To: <1364444326-16088-1-git-send-email-zhenhua.luo@freescale.com> X-Originating-IP: [128.224.163.154] Subject: Re: [oe-core v3] rpm: split out run-postinsts X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Apr 2013 08:12:50 -0000 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit On 03/28/2013 12:18 PM, Zhenhua Luo wrote: > 1. Split out run-postinsts script into separated package, sometimes only the > postinsts script is required to run all postinsts scripts in /etc/rpm-postinsts/ > instead of the whole rpm package. > 2. Set ROOTFS_PKGMANAGE_BOOTSTRAP to rpm-postinsts > > Signed-off-by: Zhenhua Luo > --- > meta/classes/rootfs_rpm.bbclass | 4 +--- > meta/recipes-devtools/rpm/rpm_5.4.9.bb | 8 +++++--- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/meta/classes/rootfs_rpm.bbclass b/meta/classes/rootfs_rpm.bbclass > index 8392a1e..07ac0e3 100644 > --- a/meta/classes/rootfs_rpm.bbclass > +++ b/meta/classes/rootfs_rpm.bbclass > @@ -3,6 +3,7 @@ > # > > ROOTFS_PKGMANAGE = "rpm smartpm" > +ROOTFS_PKGMANAGE_BOOTSTRAP = "rpm-postinsts" > > # Add 50Meg of extra space for Smart > IMAGE_ROOTFS_EXTRA_SPACE_append = "${@base_contains("PACKAGE_INSTALL", "smartpm", " + 51200", "" ,d)}" > @@ -10,9 +11,6 @@ IMAGE_ROOTFS_EXTRA_SPACE_append = "${@base_contains("PACKAGE_INSTALL", "smartpm" > # Smart is python based, so be sure python-native is available to us. > EXTRANATIVEPATH += "python-native" > > -# Postinstalls on device are handled within this class at present > -ROOTFS_PKGMANAGE_BOOTSTRAP = "" > - > do_rootfs[depends] += "rpm-native:do_populate_sysroot" > do_rootfs[depends] += "rpmresolve-native:do_populate_sysroot" > do_rootfs[depends] += "python-smartpm-native:do_populate_sysroot" > diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb b/meta/recipes-devtools/rpm/rpm_5.4.9.bb > index 6286771..cfdc390 100644 > --- a/meta/recipes-devtools/rpm/rpm_5.4.9.bb > +++ b/meta/recipes-devtools/rpm/rpm_5.4.9.bb > @@ -43,7 +43,7 @@ LICENSE = "LGPLv2.1" > LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1" > > DEPENDS = "libpcre attr acl popt ossp-uuid file bison-native" > -PR = "r61" > +PR = "r62" > > # rpm2cpio is a shell script, which is part of the rpm src.rpm. It is needed > # in order to extract the distribution SRPM into a format we can extract... > @@ -199,7 +199,7 @@ CFLAGS_append = " -DRPM_VENDOR_WINDRIVER -DRPM_VENDOR_POKY -DRPM_VENDOR_OE" > > LDFLAGS_append_libc-uclibc = "-lrt -lpthread" > > -PACKAGES = "${PN}-dbg ${PN} ${PN}-doc ${PN}-libs ${PN}-dev ${PN}-staticdev ${PN}-common ${PN}-build python-rpm-dbg python-rpm-staticdev python-rpm-dev python-rpm perl-module-rpm perl-module-rpm-dev ${PN}-locale" > +PACKAGES = "${PN}-dbg ${PN} ${PN}-doc ${PN}-libs ${PN}-dev ${PN}-staticdev ${PN}-common ${PN}-build python-rpm-dbg python-rpm-staticdev python-rpm-dev python-rpm perl-module-rpm perl-module-rpm-dev ${PN}-locale ${PN}-postinsts" > > SOLIBS = "5.4.so" > > @@ -223,9 +223,11 @@ FILES_${PN} = "${bindir}/rpm \ > ${libdir}/rpm/bin/wget \ > /var/lib/rpm \ > /var/cache/rpm \ > - ${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts \ > " > > +FILES_${PN}-postinsts = "${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts \ > + " > + > FILES_${PN}-dbg += "${libdir}/rpm/.debug \ > ${libdir}/rpm/bin/.debug \ > " Hi Luo, This patch introduces a new problem, that is, rpm-postinsts is not installed on sato images. In fact, for now, rpm-postinsts will not be installed if 'package-management' is in IMAGE_FEATURES. I've opened a bug for this problem (https://bugzilla.yoctoproject.org/show_bug.cgi?id=4160). Comments and suggestions are appreciated. Best Regards, Chen Qi