* [PATCH] cooker: now that we use a Pool, raise the exceptions
@ 2013-02-25 18:48 Christopher Larson
2013-02-26 15:29 ` Richard Purdie
0 siblings, 1 reply; 2+ messages in thread
From: Christopher Larson @ 2013-02-25 18:48 UTC (permalink / raw)
To: bitbake-devel; +Cc: Christopher Larson
From: Christopher Larson <chris_larson@mentor.com>
Multiprocessing catches these and handles passing them between processes
itself, we don't need to do it ourselves anymore.
[YOCTO #3926]
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
---
bitbake/lib/bb/cooker.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index 9f7121f..1b84e86 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -1563,12 +1563,12 @@ def parse_file((filename, appends, caches_array)):
tb = sys.exc_info()[2]
exc.recipe = filename
exc.traceback = list(bb.exceptions.extract_traceback(tb, context=3))
- return True, exc
+ raise
# Need to turn BaseExceptions into Exceptions here so we gracefully shutdown
# and for example a worker thread doesn't just exit on its own in response to
# a SystemExit event for example.
except BaseException as exc:
- return True, ParsingFailure(exc, filename)
+ raise ParsingFailure(exc, filename)
class CookerParser(object):
def __init__(self, cooker, filelist, masked):
--
1.8.1.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] cooker: now that we use a Pool, raise the exceptions
2013-02-25 18:48 [PATCH] cooker: now that we use a Pool, raise the exceptions Christopher Larson
@ 2013-02-26 15:29 ` Richard Purdie
0 siblings, 0 replies; 2+ messages in thread
From: Richard Purdie @ 2013-02-26 15:29 UTC (permalink / raw)
To: Christopher Larson; +Cc: Christopher Larson, bitbake-devel
On Mon, 2013-02-25 at 11:48 -0700, Christopher Larson wrote:
> From: Christopher Larson <chris_larson@mentor.com>
>
> Multiprocessing catches these and handles passing them between processes
> itself, we don't need to do it ourselves anymore.
>
> [YOCTO #3926]
>
> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> ---
> bitbake/lib/bb/cooker.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Makes sense, thanks (and merged)!
Richard
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-02-26 15:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-25 18:48 [PATCH] cooker: now that we use a Pool, raise the exceptions Christopher Larson
2013-02-26 15:29 ` Richard Purdie
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.