[OE-core] [PATCH] dpkg, opkg, rpm-postinst: fix overwriting the run-postinstall script
ChenQi
Qi.Chen at windriver.com
Thu Apr 11 07:06:36 UTC 2013
On 04/11/2013 02:42 PM, Laurentiu Palcu wrote:
> If multiple package managers are installed in the image, they will
> overwrite each other's run-postinsts script, resulting in postinstalls
> not beeing run at all at first boot.
>
> What this patch does:
> * checks whether opkg/dpks/rpm is actually used to install
> the packages and, only after, creates the run-postinsts script;
> * brings dpkg recipe in sync with opkg: moves the script creation from
> do_install to postinstall;
> * move creation of run-postinsts script (rpm-postinsts recipe) to the
> postinstall scriptlet in order to better control the creation of the
> script according to the package manager used;
>
> Signed-off-by: Laurentiu Palcu <laurentiu.palcu at intel.com>
> ---
> meta/recipes-devtools/dpkg/dpkg.inc | 23 ++++++++++++++---------
> meta/recipes-devtools/opkg/opkg.inc | 3 ++-
> meta/recipes-devtools/rpm/rpm-postinsts.bb | 19 ++++++++++++-------
> 3 files changed, 28 insertions(+), 17 deletions(-)
>
> diff --git a/meta/recipes-devtools/dpkg/dpkg.inc b/meta/recipes-devtools/dpkg/dpkg.inc
> index 6bb1e16..f7f7b01 100644
> --- a/meta/recipes-devtools/dpkg/dpkg.inc
> +++ b/meta/recipes-devtools/dpkg/dpkg.inc
> @@ -39,15 +39,6 @@ POSTLOG ?= "/var/log/postinstall.log"
> REDIRECT_CMD = "${@base_contains('IMAGE_FEATURES', 'debug-tweaks', '>${POSTLOG} 2>&1', '', d)}"
>
> DPKG_INIT_POSITION ?= "98"
> -do_install_prepend () {
> - install -d ${D}/${sysconfdir}/rcS.d
> - # this happens at S98 where our good 'ole packages script used to run
> - echo "#!/bin/sh
> -dpkg --configure -a ${REDIRECT_CMD}
> -rm -f ${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> -" > ${D}/${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> - chmod 0755 ${D}/${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> -}
>
> do_install_append () {
> if [ "${PN}" = "dpkg-native" ]; then
> @@ -67,6 +58,20 @@ do_install_append_class-native () {
> done
> }
>
> +pkg_postinst_${PN} () {
> +#!/bin/sh
> +if [ "x$D" != "x" ] && [ "deb" = "${IMAGE_PKGTYPE}" ]; then
> + install -d ${D}/${sysconfdir}/rcS.d
> +
> + # this happens at S98 where our good 'ole packages script used to run
> + echo "#!/bin/sh
> +dpkg --configure -a ${REDIRECT_CMD}
> +rm -f ${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> +" > ${D}/${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> + chmod 0755 ${D}/${sysconfdir}/rcS.d/S${DPKG_INIT_POSITION}run-postinsts
> +fi
> +}
> +
> PROV = "virtual/update-alternatives"
> PROV_class-native = ""
>
> diff --git a/meta/recipes-devtools/opkg/opkg.inc b/meta/recipes-devtools/opkg/opkg.inc
> index f9c1202..d400dcd 100644
> --- a/meta/recipes-devtools/opkg/opkg.inc
> +++ b/meta/recipes-devtools/opkg/opkg.inc
> @@ -62,8 +62,9 @@ REDIRECT_CMD = "${@base_contains('IMAGE_FEATURES', 'debug-tweaks', '>${POSTLOG}
>
> pkg_postinst_${PN} () {
> #!/bin/sh
> -if [ "x$D" != "x" ]; then
> +if [ "x$D" != "x" ] && [ "ipk" = "${IMAGE_PKGTYPE}" ]; then
> install -d $D${sysconfdir}/rcS.d
> +
> # this happens at S98 where our good 'ole packages script used to run
> echo "#!/bin/sh
> opkg-cl configure ${REDIRECT_CMD}
> diff --git a/meta/recipes-devtools/rpm/rpm-postinsts.bb b/meta/recipes-devtools/rpm/rpm-postinsts.bb
> index fb05ad6..281cb14 100644
> --- a/meta/recipes-devtools/rpm/rpm-postinsts.bb
> +++ b/meta/recipes-devtools/rpm/rpm-postinsts.bb
> @@ -27,15 +27,17 @@ do_compile() {
> }
>
> do_install() {
> - install -d ${D}/${sysconfdir}/rcS.d
> - # Stop $i getting expanded below...
> - i=\$i
> - cat > ${D}${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts << EOF
> -#!/bin/sh
> + :
> +}
>
> +pkg_postinst_${PN} () {
> +if [ "x$D" != "x" ] && [ "rpm" = "${IMAGE_PKGTYPE}" ]; then
> + install -d $D/${sysconfdir}/rcS.d
> + cat > $D${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts << "EOF"
> +#!/bin/sh
> . /etc/default/rcS
>
> -[ -d /etc/rpm-postinsts ] && for i in \`ls /etc/rpm-postinsts/ \`; do
> +[ -d /etc/rpm-postinsts ] && for i in `ls /etc/rpm-postinsts/`; do
> i=/etc/rpm-postinsts/$i
> echo "Running postinst $i..."
> if [ -f $i ] && $i ${REDIRECT_CMD}; then
> @@ -46,5 +48,8 @@ do_install() {
> done
> rm -f ${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts 2>/dev/null
> EOF
> - chmod 0755 ${D}${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts
> + chmod 0755 $D${sysconfdir}/rcS.d/S${POSTINSTALL_INITPOSITION}run-postinsts
> +fi
> }
> +
> +ALLOW_EMPTY_${PN} = "1"
bug#4231 (https://bugzilla.yoctoproject.org/show_bug.cgi?id=4231)?
More information about the Openembedded-core
mailing list