mirror of
https://git.yoctoproject.org/poky
synced 2026-04-18 21:32:12 +02:00
bblayers/makesetup: raise exceptions when errors happen
Otherwise the calling code can only issue a generic, unhelpful erorr message, and it's difficult to tell what went wrong if logger.error output is obscured or redirected. (From OE-Core rev: 155fe6bda2d715aa616fa8db922dbaa11ebb8080) Signed-off-by: Alexander Kanavin <alex@linutronix.de> Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
2490d0c12c
commit
f82c933a4b
@@ -48,8 +48,9 @@ class MakeSetupPlugin(LayerPlugin):
|
||||
if l_name == 'workspace':
|
||||
continue
|
||||
if l_ismodified:
|
||||
logger.error("Layer {name} in {path} has uncommitted modifications or is not in a git repository.".format(name=l_name,path=l_path))
|
||||
return
|
||||
e = "Layer {name} in {path} has uncommitted modifications or is not in a git repository.".format(name=l_name,path=l_path)
|
||||
logger.error(e)
|
||||
raise Exception(e)
|
||||
repo_path = oe.buildcfg.get_metadata_git_toplevel(l_path)
|
||||
|
||||
if self._is_submodule(repo_path):
|
||||
@@ -63,8 +64,9 @@ class MakeSetupPlugin(LayerPlugin):
|
||||
if repo_path == destdir_repo:
|
||||
repos[repo_path]['contains_this_file'] = True
|
||||
if not repos[repo_path]['git-remote']['remotes'] and not repos[repo_path]['contains_this_file']:
|
||||
logger.error("Layer repository in {path} does not have any remotes configured. Please add at least one with 'git remote add'.".format(path=repo_path))
|
||||
return
|
||||
e = "Layer repository in {path} does not have any remotes configured. Please add at least one with 'git remote add'.".format(path=repo_path)
|
||||
logger.error(e)
|
||||
raise Exception(e)
|
||||
|
||||
top_path = os.path.commonpath([os.path.dirname(r) for r in repos.keys()])
|
||||
|
||||
|
||||
Reference in New Issue
Block a user