All of lore.kernel.org
 help / color / mirror / Atom feed
* update-alternatives: Fix for compatibility with multilib recipes
@ 2012-04-14 17:31 Richard Purdie
  2012-04-14 22:20 ` Chris Larson
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Purdie @ 2012-04-14 17:31 UTC (permalink / raw)
  To: openembedded-core

The PN == BPN test was designed to exclude native/nativesdk but also unfortunately
excludes multilib unintentionally. This change fixes this and allows multilib images
with alternatives to funciton correctly.

[YOCTO #2214]

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/classes/update-alternatives.bbclass b/meta/classes/update-alternatives.bbclass
index ae58344..7b0518d 100644
--- a/meta/classes/update-alternatives.bbclass
+++ b/meta/classes/update-alternatives.bbclass
@@ -69,15 +69,16 @@ done
 }
 
 update_alternatives_batch_doinstall() {
-if [ "${PN}" = "${BPN}" ] ; then
 	for link in ${ALTERNATIVE_LINKS}
 	do
 		mv ${D}${link} ${D}${link}.${PN}
 	done
-fi
 }
 
 def update_alternatives_after_parse(d):
+    if bb.data.inherits_class('native', d) or bb.data.inherits_class('nativesdk', d):
+        return
+
     if d.getVar('ALTERNATIVE_LINKS') != None:
         doinstall = d.getVar('do_install', 0)
         doinstall += d.getVar('update_alternatives_batch_doinstall', 0)





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: update-alternatives: Fix for compatibility with multilib recipes
  2012-04-14 17:31 update-alternatives: Fix for compatibility with multilib recipes Richard Purdie
@ 2012-04-14 22:20 ` Chris Larson
  0 siblings, 0 replies; 2+ messages in thread
From: Chris Larson @ 2012-04-14 22:20 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Sat, Apr 14, 2012 at 10:31 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>  def update_alternatives_after_parse(d):
> +    if bb.data.inherits_class('native', d) or bb.data.inherits_class('nativesdk', d):
> +        return
> +

Not an issue, but wanted to point out we have a utility function for
this sort of thing, for future reference. bb.utils.inherits(d,
'native', 'nativesdk'), iirc.
-- 
Christopher Larson



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-04-14 22:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-14 17:31 update-alternatives: Fix for compatibility with multilib recipes Richard Purdie
2012-04-14 22:20 ` Chris Larson

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.