All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Shane Wang <shane.wang@intel.com>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH 5/8] runqueue.py: check results[0] in keys of build_pids before being used to avoid exceptions
Date: Thu, 29 Mar 2012 21:04:09 +0100	[thread overview]
Message-ID: <1333051449.18082.10.camel@ted> (raw)
In-Reply-To: <b0f25a4ae7b257c5e0631a3e5c1f90facf25aca6.1333025491.git.shane.wang@intel.com>

On Thu, 2012-03-29 at 20:54 +0800, Shane Wang wrote:
> 
> 
> ne.wang@intel.com>
>                                To: 
> bitbake-devel@lists.openembedded.org
>                           Subject: 
> [bitbake-devel] [PATCH 5/8]
> runqueue.py: check results[0] in
> keys of build_pids before being
> used to avoid exceptions
>                              Date: 
> Thu, 29 Mar 2012 20:54:54 +0800
> (29/03/12 13:54:54)
> 
> 
> [Yocto #2186]
> 
> Signed-off-by: Shane Wang <shane.wang@intel.com>
> ---
>  bitbake/lib/bb/runqueue.py |   20 ++++++++++++--------
>  1 files changed, 12 insertions(+), 8 deletions(-)

This kind of change sets off alarm bells. The big question is why are
you seeing exceptions? I suspect you're forking off processes within hob
which are then confusing the waitpid code. I'd have to ask why the UI is
forking processes when a build is running and why we're suddenly started
seeing this...

So can you please explain the problem further so we can fix the real
problem? I did look at #2186 but that doesn't help me either :(

Cheers,

Richard




  reply	other threads:[~2012-03-29 20:13 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-29 12:54 [PATCH 0/8] Hob: bug fixes Shane Wang
2012-03-29 12:54 ` [PATCH 1/8] Hob: clear the building state after a successful build, as a failed build does Shane Wang
2012-03-29 12:54 ` [PATCH 2/8] Hob: Change icon to error because parsing error is an error Shane Wang
2012-03-29 12:54 ` [PATCH 3/8] Hob: Change "Just bake" to "Build image" as required Shane Wang
2012-03-29 12:54 ` [PATCH 4/8] Hob: make ORANGE to be the same as the color on the dialog Shane Wang
2012-03-29 12:54 ` [PATCH 5/8] runqueue.py: check results[0] in keys of build_pids before being used to avoid exceptions Shane Wang
2012-03-29 20:04   ` Richard Purdie [this message]
2012-03-30  6:10     ` Wang, Shane
2012-03-30  6:15       ` Chris Larson
2012-03-30  9:27       ` Richard Purdie
2012-03-30  6:11     ` Wang, Shane
2012-03-29 12:54 ` [PATCH 6/8] Hob: some color style changes to make HobTabBar more close to visual design Shane Wang
2012-03-29 12:54 ` [PATCH 7/8] Hob: fixed the 'error' icon source picture and adjusted corresponding showed icon size Shane Wang
2012-03-29 12:54 ` [PATCH 8/8] Hob: in build details page, change error icon to denied icon Shane Wang
2012-03-29 20:26 ` [PATCH 0/8] Hob: bug fixes Richard Purdie

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=1333051449.18082.10.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=bitbake-devel@lists.openembedded.org \
    --cc=shane.wang@intel.com \
    /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.