* [oe-core v3] rpm: split out run-postinsts
@ 2013-03-28 4:18 Zhenhua Luo
2013-04-01 7:55 ` ChenQi
0 siblings, 1 reply; 3+ messages in thread
From: Zhenhua Luo @ 2013-03-28 4:18 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenhua Luo
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 <zhenhua.luo@freescale.com>
---
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 \
"
--
1.7.5.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [oe-core v3] rpm: split out run-postinsts
2013-03-28 4:18 [oe-core v3] rpm: split out run-postinsts Zhenhua Luo
@ 2013-04-01 7:55 ` ChenQi
2013-04-03 0:31 ` Saul Wold
0 siblings, 1 reply; 3+ messages in thread
From: ChenQi @ 2013-04-01 7:55 UTC (permalink / raw)
To: openembedded-core
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 <zhenhua.luo@freescale.com>
> ---
> 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
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [oe-core v3] rpm: split out run-postinsts
2013-04-01 7:55 ` ChenQi
@ 2013-04-03 0:31 ` Saul Wold
0 siblings, 0 replies; 3+ messages in thread
From: Saul Wold @ 2013-04-03 0:31 UTC (permalink / raw)
To: Zhenhua Luo; +Cc: openembedded-core
On 04/01/2013 12:55 AM, ChenQi wrote:
> 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.
This change seems to draw all the RPM dependencies into the package even
if package-management is not added. We are building the poky-tiny
distro with core-image-basic and icu and other dependencies of RPM are
getting pulled in when the should not be.
ICU shows this on the autobuilder since we tweak eglibc to remove bits.
Please review this patch, change as needed and test with
DISTRO="poky-tiny" bitbake core-image-minimal
This is already in master, so we need a patch ASAP! I will be filing a
bug #4175.
Thanks
Sau!
>> 2. Set ROOTFS_PKGMANAGE_BOOTSTRAP to rpm-postinsts
>>
>> Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
>> ---
>> 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
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-04-03 0:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-28 4:18 [oe-core v3] rpm: split out run-postinsts Zhenhua Luo
2013-04-01 7:55 ` ChenQi
2013-04-03 0:31 ` Saul Wold
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox