From 39fffe33b8c9dc5e4349a397c964632cb66a41d0 Mon Sep 17 00:00:00 2001 From: Russ Allbery Date: Thu, 12 Feb 2009 17:27:03 -0800 Subject: [PATCH] Don't exit successfully if sourcing the debconf module fails * Abort the config script rather than exiting successfully if the debconf shell module is not available. The config script should never be run in the absence of debconf, so if this fails, something deeper is wrong and we shouldn't blindly continue. --- debian/changelog | 4 ++++ debian/openafs-client.config | 2 +- debian/openafs-fileserver.config | 2 +- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/debian/changelog b/debian/changelog index eb6869aea..8b17f710d 100644 --- a/debian/changelog +++ b/debian/changelog @@ -44,6 +44,10 @@ openafs (1.4.8.dfsg1-1) UNRELEASED; urgency=low default command stacks in dh. - Use a debian/clean configuration file to let dh clean do more of the work rather than using rm commands in debian/rules. + * Abort the config script rather than exiting successfully if the + debconf shell module is not available. The config script should never + be run in the absence of debconf, so if this fails, something deeper + is wrong and we shouldn't blindly continue. * Add ${misc:Depends} to all dependencies. * Add explanations for the Lintian overrides. * Use set -e instead of #!/bin/sh -e in maintainer scripts. diff --git a/debian/openafs-client.config b/debian/openafs-client.config index 8c100163a..759bbb2f0 100644 --- a/debian/openafs-client.config +++ b/debian/openafs-client.config @@ -1,7 +1,7 @@ #!/bin/sh set -e -. /usr/share/debconf/confmodule || exit +. /usr/share/debconf/confmodule db_version 2.0 if [ -r /etc/openafs/ThisCell ] ; then diff --git a/debian/openafs-fileserver.config b/debian/openafs-fileserver.config index ed7300727..04e326008 100644 --- a/debian/openafs-fileserver.config +++ b/debian/openafs-fileserver.config @@ -1,7 +1,7 @@ #!/bin/sh set -e -. /usr/share/debconf/confmodule || exit +. /usr/share/debconf/confmodule db_version 2.0 -- 2.39.5