diff options
| author |   Paul Eggleton <paul.eggleton@linux.intel.com> | 2017-02-07 09:45:43 +1300 |
|---|---|---|
| committer |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-02-07 14:48:55 +0000 |
| commit | b9592bd3c1ab2155b9f62d18ce593b9c88eff279 (patch) | |
| tree | 802637a407c34fd44ab9476f6ae6ce442bf98c10 /scripts/lib/devtool/standard.py | |
| parent | 3427508b6ce865654f8bf01a6fc04b83c70315d3 (diff) | |
| download | openembedded-core-contrib-b9592bd3c1ab2155b9f62d18ce593b9c88eff279.tar.gz | |
devtool: improve parse failure handling
With the move to tinfoil2, the behaviour when parsing failed has changed
a bit - exceptions are now raised, so handle these appropriately.
Specifically when if parsing the recipe created when running devtool add
fails, rename it to .bb.parsefailed so that the user can run bitbake
afterwards without parsing being interrupted.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/devtool/standard.py')
| -rw-r--r-- | scripts/lib/devtool/standard.py | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py index 30b247fa74..5bd498c933 100644 --- a/scripts/lib/devtool/standard.py +++ b/scripts/lib/devtool/standard.py @@ -224,8 +224,17 @@ def add(args, config, basepath, workspace): tinfoil = setup_tinfoil(config_only=True, basepath=basepath) try: - rd = tinfoil.parse_recipe_file(recipefile, False) + try: + rd = tinfoil.parse_recipe_file(recipefile, False) + except Exception as e: + logger.error(str(e)) + rd = None if not rd: + # Parsing failed. We just created this recipe and we shouldn't + # leave it in the workdir or it'll prevent bitbake from starting + movefn = '%s.parsefailed' % recipefile + logger.error('Parsing newly created recipe failed, moving recipe to %s for reference. If this looks to be caused by the recipe itself, please report this error.' % movefn) + shutil.move(recipefile, movefn) return 1 if args.fetchuri and not args.no_git: |