From 332d2a19220c10c0faa48ef0a973c8361a18a231 Mon Sep 17 00:00:00 2001 From: Allan McRae Date: Sat, 30 Sep 2023 23:27:04 +1000 Subject: [PATCH] repo-add: clarify exit messages Clarify if repo-add does not create a new database due to failures or due to there being nothing to do. Signed-off-by: Allan McRae --- scripts/repo-add.sh.in | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in index 0ec998ad..b3e85e6a 100644 --- a/scripts/repo-add.sh.in +++ b/scripts/repo-add.sh.in @@ -700,13 +700,17 @@ for arg in "${args[@]:1}"; do done # if the whole operation was a success, re-zip and rotate databases -if (( !fail )) && (( DB_MODIFIED )); then +if (( fail )); then + msg "$(gettext "Package database was not modified due to errors.")" + exit 1 +fi + +if (( DB_MODIFIED )); then msg "$(gettext "Creating updated database file '%s'")" "$REPO_DB_FILE" create_db rotate_db else - msg "$(gettext "No packages modified, nothing to do.")" - exit $(( fail != 0 )) + msg "$(gettext "No changes made to package database.")" fi exit 0