Skip to content

Commit 96c8002

Browse files
authored
Merge pull request aurutils#1219 from aurutils/build-partial-warning
Revert "build: error on partially built package group"
2 parents bcd3794 + a4b557d commit 96c8002

File tree

1 file changed

+5
-10
lines changed

1 file changed

+5
-10
lines changed

lib/pacman/aur-build

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -382,21 +382,16 @@ while IFS= read "${read_args[@]}" -ru "$fd" path; do
382382
printf '%q\n' >&2 "${exists[@]}"
383383
create_package=0
384384

385-
if [[ -v results_file ]]; then
386-
printf "exist:file://%s\n" "${exists[@]}" | tee -a "$results_file" >/dev/null
387-
fi
388-
389385
elif (( ${#exists[@]} )); then
390386
# Since `makepkg` does not allow building split packages
391-
# individually, we require a new build when part of the
387+
# individually, we restart the build when part of the
392388
# package group is unavailable (including -debug packages)
393-
printf >&2 '%s: error: package group partially built\n' "$argv0"
389+
printf >&2 '%s: warning: package group partially built\n' "$argv0"
394390
printf '%q\n' >&2 "${exists[@]}"
391+
fi
395392

396-
# Rebuilding the package group at this stage may result in
397-
# checksum errors, if part of the package group is
398-
# available in pacman's cache, so we exit with an error.
399-
exit 2
393+
if (( ${#exists[@]} )) && [[ -v results_file ]]; then
394+
printf "exist:file://%s\n" "${exists[@]}" | tee -a "$results_file" >/dev/null
400395
fi
401396
fi
402397

0 commit comments

Comments
 (0)