All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Ferguson <thoughtmonster@gmail.com>
To: openembedded-devel@lists.openembedded.org
Cc: Alex Ferguson <thoughtmonster@gmail.com>
Subject: [PATCH] classes: Modify rootfs_ipk.bbclass for -force-overwrite.
Date: Fri, 29 Oct 2010 17:08:54 +0300	[thread overview]
Message-ID: <1288361334-17427-1-git-send-email-thoughtmonster@gmail.com> (raw)

Rationale: In Jlime we ship a package called jlime-extras which
contains several files (configuration files, scripts, etc.), some
of which conflict with files provided by other packages. This is
intentional, and is for distro customization purposes. For example
we ship a modified version of the /usr/bin/startx script, some
alternative icons for applications, some modified .desktop files
and so on. Building an image with this jlime-extras package fails,
of course, due to conflicting files.

To overcome this issue, I have added a base-conditional to
rootfs_ipk.bbclass, adding the PACKAGE_FORCE_OVERWRITE variable
to IPKG_ARGS which enables -force-overwrite for opkg, and which
can be used in image recipes individually without affecting others.

I believe this functionality might be useful for everyone and comes
at almost no additional cost, which is why I'm sending this patch
for consideration by the OE developers.

I'd be more than happy to help in any way in order to get this in.

Thank you.

Signed-off-by: Alex Ferguson <thoughtmonster@gmail.com>
---
 classes/rootfs_ipk.bbclass |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/classes/rootfs_ipk.bbclass b/classes/rootfs_ipk.bbclass
index a4a8563..dd8586a 100644
--- a/classes/rootfs_ipk.bbclass
+++ b/classes/rootfs_ipk.bbclass
@@ -9,9 +9,10 @@ do_rootfs[depends] += "opkg-native:do_populate_sysroot"
 do_rootfs[lockfiles] = "${DEPLOY_DIR_IPK}.lock"
 
 IPKG_TMP_DIR = "${IMAGE_ROOTFS}-tmp"
-IPKG_ARGS = "-f ${IPKGCONF_TARGET} -o ${IMAGE_ROOTFS} -t ${IPKG_TMP_DIR} ${@base_conditional("PACKAGE_INSTALL_NO_DEPS", "1", "-nodeps", "", d)}"
+IPKG_ARGS = "-f ${IPKGCONF_TARGET} -o ${IMAGE_ROOTFS} -t ${IPKG_TMP_DIR} ${@base_conditional("PACKAGE_INSTALL_NO_DEPS", "1", "-nodeps", "", d)} ${@base_conditional("PACKAGE_FORCE_OVERWRITE", "1", "-force-overwrite", "", d)}"
 
 PACKAGE_INSTALL_NO_DEPS ?= "0"
+PACKAGE_FORCE_OVERWRITE ?= "0"
 
 # What support to provide for online management of packages at run time?
 #  full -> traditional system, opkg is installed with all metadata
-- 
1.7.3.2




             reply	other threads:[~2010-10-29 14:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-29 14:08 Alex Ferguson [this message]
2010-10-29 14:21 ` [PATCH] classes: Modify rootfs_ipk.bbclass for -force-overwrite Eric Bénard
2010-10-29 14:39 ` Graeme Gregory
2010-10-29 14:55 ` Martin Jansa
2010-10-29 15:40   ` Alex Ferguson
2010-10-29 17:44 ` Khem Raj

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=1288361334-17427-1-git-send-email-thoughtmonster@gmail.com \
    --to=thoughtmonster@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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.