build: Bail on error
All threads resolved!
All threads resolved!
If one is missing one of the deps (e.g. yq for android devices) we should abort early instead of continuing and producing potentially confusing errors down the road.
Merge request reports
Activity
- Resolved by Arnaud Ferraris
This would need a bit more changes:
- there are a few occurences of
[ -f <file> ] && ...
which will error out if the file doesn't exist - it would be a good opportunity to drop all
|| exit 1
as they wouldn't be needed anymore
- there are a few occurences of
added 3 commits
-
e919c976...e6c6b838 - 2 commits from branch
master
- 1199b375 - build: Bail on error
-
e919c976...e6c6b838 - 2 commits from branch
- Resolved by Arnaud Ferraris
v1.2: removed
|| true
(which was added in v1.1) as it's not necessary (see !129 (comment 580423))added 5 commits
-
dda63c52...91568884 - 4 commits from branch
master
- 3ac5bca9 - build: Bail on error
-
dda63c52...91568884 - 4 commits from branch
Please register or sign in to reply