apt-key errors out nicely if wget is not installed (Closes: #545754)

diff --git a/cmdline/apt-key b/cmdline/apt-key
index 5f4e02fdf03975bdda0a02c80f0efdb46efa0593..24010edf3fb390078a275edc52ad4dd8ded58245 100755 (executable)
--- a/cmdline/apt-key
+++ b/cmdline/apt-key
@@ -56,7 +56,14 @@ add_keys_with_verify_against_master_keyring() {
 # (otherwise it does not make sense from a security POV)
 net_update() {
     if [ -z "$ARCHIVE_KEYRING_URI" ]; then
-       echo "ERROR: no location for the archive-keyring given" 
+       echo "ERROR: no location for the archive-keyring given"
+       exit 1
+    fi
+    # in theory we would need to depend on wget for this, but this feature
+    # isn't useable in debian anyway as we have no keyring uri nor a master key
+    if ! which wget >/dev/null 2>&1; then
+       echo "ERROR: an installed wget is required for a network-based update"
+       exit 1
     fi
     if [ ! -d /var/lib/apt/keyrings ]; then
        mkdir -p /var/lib/apt/keyrings

diff --git a/debian/changelog b/debian/changelog
index ffd8f5c595421130cf529ec0480faeb4e88f0695..f70c5735f9cb424290f837f877381dbadc5682f3 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -39,6 +39,8 @@ apt (0.7.26) UNRELEASED; urgency=low
   * share/*-archive.gpg:
     - remove the horrible outdated files. We already depend on
       the keyring so we don't need to ship our own version
+  * cmdline/apt-key:
+    - errors out if wget is not installed (Closes: #545754)
 
  -- Michael Vogt <mvo@debian.org>  Thu, 10 Dec 2009 22:02:38 +0100