repos: Fail on unsuccessful patching
There is no good reason to only warn about an incorrect patch entry or a missing patch file. This just causes CI systems to succeed with a build - or fail at unexpected places. Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com> Signed-off-by: Daniel Wagner <daniel.wagner@siemens.com>
This commit is contained in:
parent
2f7650bb05
commit
2f24bdeac2
26
kas/repos.py
26
kas/repos.py
@ -239,13 +239,13 @@ class RepoImpl(Repo):
|
|||||||
None)
|
None)
|
||||||
|
|
||||||
if not other_repo:
|
if not other_repo:
|
||||||
logging.warning('Could not find referenced repo. '
|
logging.error('Could not find referenced repo. '
|
||||||
'(missing repo: %s, repo: %s, '
|
'(missing repo: %s, repo: %s, '
|
||||||
'patch entry: %s)',
|
'patch entry: %s)',
|
||||||
patch['repo'],
|
patch['repo'],
|
||||||
self.name,
|
self.name,
|
||||||
patch['id'])
|
patch['id'])
|
||||||
continue
|
return 1
|
||||||
|
|
||||||
path = os.path.join(other_repo.path, patch['path'])
|
path = os.path.join(other_repo.path, patch['path'])
|
||||||
cmd = []
|
cmd = []
|
||||||
@ -255,12 +255,12 @@ class RepoImpl(Repo):
|
|||||||
elif os.path.isdir(path):
|
elif os.path.isdir(path):
|
||||||
cmd = self.apply_patches_quilt_cmd(path)
|
cmd = self.apply_patches_quilt_cmd(path)
|
||||||
else:
|
else:
|
||||||
logging.warning('Could not find patch. '
|
logging.error('Could not find patch. '
|
||||||
'(patch path: %s, repo: %s, patch entry: %s)',
|
'(patch path: %s, repo: %s, patch entry: %s)',
|
||||||
path,
|
path,
|
||||||
self.name,
|
self.name,
|
||||||
patch['id'])
|
patch['id'])
|
||||||
continue
|
return 1
|
||||||
|
|
||||||
(retc, output) = yield from run_cmd_async(cmd,
|
(retc, output) = yield from run_cmd_async(cmd,
|
||||||
cwd=self.path,
|
cwd=self.path,
|
||||||
|
Loading…
Reference in New Issue
Block a user