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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox