From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 4/6 v2] package_rpm: Update the way the multilib package names are translated
Date: Fri, 7 Dec 2012 15:00:06 -0600 [thread overview]
Message-ID: <50C258D6.4020506@windriver.com> (raw)
In-Reply-To: <1354914756-9488-1-git-send-email-mark.hatle@windriver.com>
Fix a minor mistake in the original patch...
Original:
+ sdk_archs=${SDK_PACKAGE_ARCH}
new:
+ sdk_archs="${SDK_PACKAGE_ARCHS}"
Somehow I must have cleaned up something at the last minute and forgotten to
verify the SDK_PACKAGE_ARCHS worked correctly. Not only was there a typo, but
missing ". Fixed now and verified locally.
--Mark
On 12/7/12 3:12 PM, Mark Hatle wrote:
> The variable MULTILIB_PACKAGE_ARCHS has been removed in favor of a
> repurposed MULTILIB_PREFIX_LIST. The format of this item is now
> <libid>:<arch>:<arch1>:...:<archN>. This ensures that we can correctly
> translate the libid to one of the supported archs in a tri-lib system.
>
> All of the users of MULTILIB_PREFIX_LIST and MULTILIB_PACKAGE_ARCHS have
> been modified accordingly.
>
> Also change the way attempted packages are installed, verify the package
> exists in the translate functions, then perform the install in one single
> operation. This results in a significantly faster install time.
>
> Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
> ---
> meta/classes/package_rpm.bbclass | 167 ++++++++++++++++++++++++++++++---
> meta/classes/populate_sdk_rpm.bbclass | 26 ++++--
> meta/classes/rootfs_rpm.bbclass | 47 +++++-----
> 3 files changed, 195 insertions(+), 45 deletions(-)
>
> diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass
> index 4b81b68..091880e 100644
> --- a/meta/classes/package_rpm.bbclass
> +++ b/meta/classes/package_rpm.bbclass
> @@ -24,11 +24,25 @@ package_update_index_rpm () {
> return
> fi
>
> - base_archs="`echo ${PACKAGE_ARCHS} | sed 's/-/_/g'`"
> - ml_archs="`echo ${MULTILIB_PACKAGE_ARCHS} | sed 's/-/_/g'`"
> - sdk_archs="`echo ${SDK_PACKAGE_ARCHS} | sed 's/-/_/g'`"
> + sdk_archs="${SDK_PACKAGE_ARCHS}"
> + sdk_archs=${sdk_archs//-/_}
> +
....
prev parent reply other threads:[~2012-12-07 21:14 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-07 21:12 [PATCH 4/6 v2] package_rpm: Update the way the multilib package names are translated Mark Hatle
2012-12-07 21:00 ` Mark Hatle [this message]
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=50C258D6.4020506@windriver.com \
--to=mark.hatle@windriver.com \
--cc=openembedded-core@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.