From: Mark Hatle <mark.hatle@windriver.com>
To: Saul Wold <sgw@linux.intel.com>
Cc: B40290@freescale.com, Zhenhua Luo <zhenhua.luo@freescale.com>,
openembedded-core@lists.openembedded.org
Subject: Re: rpm: split out run-postinsts
Date: Wed, 20 Mar 2013 13:57:56 -0500 [thread overview]
Message-ID: <514A06B4.1080302@windriver.com> (raw)
In-Reply-To: <514A0573.2060809@linux.intel.com>
On 3/20/13 1:52 PM, Saul Wold wrote:
> On 03/20/2013 02:39 AM, Zhenhua Luo wrote:
>> 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
>>
> So are you trying to build a system that uses RPM to install the initial
> rootfs, but you don't want RPM actually installed?
That seems reasonable to me. I hadn't realized that it was rolled into the
target RPM package like this. In early implementations this script was
installed directly by the rootfs_rpm.
--Mark
>
>> Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
>> ---
>> meta/recipes-devtools/rpm/rpm_5.4.9.bb | 6 ++++--
>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
>> index 6286771..896e3a4 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...
>> @@ -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 \
>> + "
>> +
>
> This does not look right to me, you seem to be missing a PACKAGES entry
> for the postinsts package. So this really won't do anything.
>
> Sau!
>
>
>> FILES_${PN}-dbg += "${libdir}/rpm/.debug \
>> ${libdir}/rpm/bin/.debug \
>> "
>>
next prev parent reply other threads:[~2013-03-20 19:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-20 9:39 rpm: split out run-postinsts Zhenhua Luo
2013-03-20 18:52 ` Saul Wold
2013-03-20 18:57 ` Mark Hatle [this message]
2013-03-21 1:57 ` Luo Zhenhua-B19537
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=514A06B4.1080302@windriver.com \
--to=mark.hatle@windriver.com \
--cc=B40290@freescale.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=sgw@linux.intel.com \
--cc=zhenhua.luo@freescale.com \
/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.