Openembedded Core Discussions
 help / color / mirror / Atom feed
From: "Aníbal Limón" <anibal.limon@linux.intel.com>
To: Chen Qi <Qi.Chen@windriver.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/1] package_manager.py: fix rootfs failure with multilib enabled
Date: Thu, 29 Jan 2015 11:43:33 -0600	[thread overview]
Message-ID: <54CA7145.1090605@linux.intel.com> (raw)
In-Reply-To: <b35561e993f50d52ae592feddd907881b940f312.1422498020.git.Qi.Chen@windriver.com>

Hi Chen,

I tested the patch and is ok can contain (when merge),

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>

Also the same issue is presented in dizzy and the patch needs to be 
backported.

Best regards,
     alimon

On 28/01/15 20:22, Chen Qi wrote:
> With the current code, if we use debian package backend and enable
> multilib support, the do_rootfs process would always fail with error
> messages like below.
>
>      E: Unable to locate package packagegroup-core-boot
>
> This patch fixes the above problem.
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>   meta/lib/oe/package_manager.py | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
> index 69100f1..6f49c69 100644
> --- a/meta/lib/oe/package_manager.py
> +++ b/meta/lib/oe/package_manager.py
> @@ -1723,9 +1723,12 @@ class DpkgPM(PackageManager):
>               with open(self.d.expand("${STAGING_ETCDIR_NATIVE}/apt/apt.conf.sample")) as apt_conf_sample:
>                   for line in apt_conf_sample.read().split("\n"):
>                       match_arch = re.match("  Architecture \".*\";$", line)
> +                    architectures = ""
>                       if match_arch:
>                           for base_arch in base_arch_list:
> -                            apt_conf.write("  Architecture \"%s\";\n" % base_arch)
> +                            architectures += "\"%s\";" % base_arch
> +                        apt_conf.write("  Architectures {%s};\n" % architectures);
> +                        apt_conf.write("  Architecture \"%s\";\n" % base_archs)
>                       else:
>                           line = re.sub("#ROOTFS#", self.target_rootfs, line)
>                           line = re.sub("#APTCONF#", self.apt_conf_dir, line)



      reply	other threads:[~2015-01-29 17:43 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-29  2:22 [PATCH 0/1] package_manager.py: fix rootfs failure for package_deb with multilib enabled Chen Qi
2015-01-29  2:22 ` [PATCH 1/1] package_manager.py: fix rootfs failure " Chen Qi
2015-01-29 17:43   ` Aníbal Limón [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=54CA7145.1090605@linux.intel.com \
    --to=anibal.limon@linux.intel.com \
    --cc=Qi.Chen@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox