From: Laurentiu Palcu <laurentiu.palcu@intel.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: Richard Purdie <rpurdie@rpsys.net>,
openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] python-smartpm: really ignore conflicts during install with --attempt
Date: Mon, 10 Feb 2014 12:54:24 +0200 [thread overview]
Message-ID: <20140210105423.GD10078@lpalcu-linux> (raw)
In-Reply-To: <1526681.3gOseygUtX@peggleto-mobl5.ger.corp.intel.com>
On Mon, Feb 10, 2014 at 10:01:32AM +0000, Paul Eggleton wrote:
> On Monday 10 February 2014 09:59:49 Laurentiu Palcu wrote:
> > On Mon, Feb 10, 2014 at 07:37:12AM +0200, Laurentiu Palcu wrote:
> > > On Sun, Feb 09, 2014 at 11:25:11PM +0000, Richard Purdie wrote:
> > > > On Thu, 2014-02-06 at 13:39 +0000, Paul Eggleton wrote:
> > > > > The original patch added in OE-Core commit
> > > > > bdf07b1698d228dc7ff555199a269b1ff8ceca19 was supposed to ignore
> > > > > conflicts, but it was unable to do so because it wasn't raising errors
> > > > > in the right place. When the --attempt option is used (as is done in
> > > > > complementary package installation for RPM), raise errors immediately
> > > > > on conflicts, catch errors at the right point so that requested
> > > > > packages
> > > > > and their dependencies can be ignored, and print appropriate warnings
> > > > > when doing so.
> > > > >
> > > > > Fixes [YOCTO #5313].
> > > > >
> > > > > Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> > > > > ---
> > > > >
> > > > > .../python/python-smartpm/smart-attempt.patch | 134
> > > > > +++++++++++++++++++-- 1 file changed, 121 insertions(+), 13
> > > > > deletions(-)
> > > >
> > > > Could you take a look at:
> > > >
> > > > http://autobuilder.yoctoproject.org/main/builders/nightly-multilib/build
> > > > s/22
> > > >
> > > > please?
> > > >
> > > > I've not 100% confirmed it was this change, it could be something else
> > > > (e.g. Laurentiu's rootfs changes) but something in master-next broke
> > > > multilib :/
> > >
> > > I'll do a build locally with your master-next changes and see what
> > > happens. At first sight, it looks like an issue with the rootfs
> > > changes. However, 'smart channel --add' is not the command I was
> > > expecting to fail...
> >
> > I pushed a fix on poky-contrib:lpalcu/rootfs_refactoring_ship_oecore
> >
> > lib/oe/package_manager.py: RpmPM: fix issue with multilib builds
>
> The lack of detail in the error message is a little bit concerning - we're not
> getting the output from smart in the log, so other than the return value it's
> hard to determine what might have gone wrong. Is this something that could be
> fixed as well?
Sure, I'll prepare another patch that will print all the errors in the
log.
laurentiu
>
> Cheers,
> Paul
>
> --
>
> Paul Eggleton
> Intel Open Source Technology Centre
prev parent reply other threads:[~2014-02-10 10:54 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-06 13:39 [PATCH] python-smartpm: really ignore conflicts during install with --attempt Paul Eggleton
2014-02-09 23:25 ` Richard Purdie
2014-02-10 5:37 ` Laurentiu Palcu
2014-02-10 7:59 ` Laurentiu Palcu
2014-02-10 10:01 ` Paul Eggleton
2014-02-10 10:54 ` Laurentiu Palcu [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=20140210105423.GD10078@lpalcu-linux \
--to=laurentiu.palcu@intel.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul.eggleton@linux.intel.com \
--cc=rpurdie@rpsys.net \
/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