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 /scripts/lib | |
| 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>
Diffstat (limited to 'scripts/lib')
| -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) | 
