This reduces code duplication and also makes --populate a non-interactive 
function.

Signed-off-by: Pierre Schmitz <[email protected]>
---
 scripts/pacman-key.sh.in |    3 ++-
 1 Datei geändert, 2 Zeilen hinzugefügt(+), 1 Zeile entfernt(-)

diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in
index ae491d2..e5b6021 100644
--- a/scripts/pacman-key.sh.in
+++ b/scripts/pacman-key.sh.in
@@ -302,7 +302,7 @@ populate_keyring() {
                msg "$(gettext "Locally signing trusted keys in keyring...")"
                for key_id in "${!trusted_ids[@]}"; do
                        msg2 "$(gettext "Locally signing key %s...")" 
"${key_id}"
-                       "${GPG_PACMAN[@]}" --quiet --lsign-key "${key_id}"
+                       lsign_keys "${key_id}"
                done
                msg "$(gettext "Importing owner trust values...")"
                for keyring in "${KEYRINGIDS[@]}"; do
@@ -440,6 +440,7 @@ list_sigs() {
 
 lsign_keys() {
        check_keyids_exist
+       # we cannot use --yes here as gpg would still ask for confirmation if a 
key has more than one uid
        printf 'y\ny\n' | LANG=C "${GPG_PACMAN[@]}" --command-fd 0 --quiet 
--batch --lsign-key "$@" 2>/dev/null
        if (( PIPESTATUS[1] )); then
                error "$(gettext "A specified key could not be locally 
signed.")"
-- 
1.7.10.4

Reply via email to