diff options
author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2015-10-14 16:43:24 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-10-19 17:48:34 +0100 |
commit | a46fb1a06be2df00a0bceb4e60ebdecb55b5974c (patch) | |
tree | 09f4304d3192698f8ba6415d8694f669921b2978 | |
parent | bd8ff33cf2439536c6e41cf0ee9dd8fb3b64770a (diff) | |
download | openembedded-core-a46fb1a06be2df00a0bceb4e60ebdecb55b5974c.tar.gz openembedded-core-a46fb1a06be2df00a0bceb4e60ebdecb55b5974c.tar.bz2 openembedded-core-a46fb1a06be2df00a0bceb4e60ebdecb55b5974c.zip |
devtool: extract: fix error handling
If recipe parsing fails we need to exit out instead of attempting to use
the data object that's set to None, which resulted in a traceback.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
-rw-r--r-- | scripts/lib/devtool/standard.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py index 6ce3144dd0..5d7e903670 100644 --- a/scripts/lib/devtool/standard.py +++ b/scripts/lib/devtool/standard.py @@ -225,6 +225,9 @@ def extract(args, config, basepath, workspace): import bb tinfoil = _prep_extract_operation(config, basepath, args.recipename) + if not tinfoil: + # Error already shown + return 1 rd = parse_recipe(config, tinfoil, args.recipename, True) if not rd: @@ -271,6 +274,8 @@ def _prep_extract_operation(config, basepath, recipename): trying to extract a package. Returns the tinfoil instance to be used.""" tinfoil = setup_tinfoil(basepath=basepath) rd = parse_recipe(config, tinfoil, recipename, True) + if not rd: + return None if bb.data.inherits_class('kernel-yocto', rd): tinfoil.shutdown() @@ -500,6 +505,9 @@ def modify(args, config, basepath, workspace): args.srctree) if args.extract: tinfoil = _prep_extract_operation(config, basepath, args.recipename) + if not tinfoil: + # Error already shown + return 1 else: tinfoil = setup_tinfoil(basepath=basepath) |