From 8a5b3cfd91841c97fbc8a003857cacbd602646ed Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Fri, 16 Mar 2012 14:18:05 -0700 Subject: [PATCH] debian: Use a different way to avoid failing install without kernel module. The dh_installinit --error-handler option makes a lot of sense, but after playing with it for a while I could not figure out a nice way to use it only for openvswitch-switch without either duplicating the dh_installinit fragments in postinst and prerm (the actual bug that was reported) or omitting them for some package. Also, we forgot to write the error handler function for the prerm. This commit switches to a different way to avoid failing the install when the kernel module is not available, without using --error-handler. CC: 663051@bugs.debian.org Reported-by: Thomas Goirand Reviewed-by: Simon Horman Signed-off-by: Ben Pfaff --- debian/openvswitch-switch.init | 7 +++++++ debian/openvswitch-switch.postinst | 18 ++---------------- debian/rules | 3 +-- 3 files changed, 10 insertions(+), 18 deletions(-) diff --git a/debian/openvswitch-switch.init b/debian/openvswitch-switch.init index 98863e3b..aebf21e5 100755 --- a/debian/openvswitch-switch.init +++ b/debian/openvswitch-switch.init @@ -58,6 +58,13 @@ start () { echo "For instructions, read" fi echo "/usr/share/doc/openvswitch-datapath-source/README.Debian" + + if test X"$OVS_MISSING_KMOD_OK" = Xyes; then + # We're being invoked by the package postinst. Do not + # fail package installation just because the kernel module + # is not available. + exit 0 + fi fi set ovs_ctl ${1-start} --system-id=random if test X"$FORCE_COREFILES" != X; then diff --git a/debian/openvswitch-switch.postinst b/debian/openvswitch-switch.postinst index c50853a1..7b9d7bc8 100755 --- a/debian/openvswitch-switch.postinst +++ b/debian/openvswitch-switch.postinst @@ -44,25 +44,11 @@ case "$1" in ;; esac -HAVE_KMOD="no" - -init_script_error () { - if test X"$HAVE_KMOD" = Xno; then - exit 0 - fi - exit 1 -} - # Do not fail package installation just because the kernel module # is not available. -if test -x /etc/init.d/openvswitch-switch; then - if invoke-rc.d openvswitch-switch load-kmod; then - HAVE_KMOD="yes" - fi -fi +OVS_MISSING_KMOD_OK=yes +export OVS_MISSING_KMOD_OK #DEBHELPER# exit 0 - - diff --git a/debian/rules b/debian/rules index 4160025a..24c9850f 100755 --- a/debian/rules +++ b/debian/rules @@ -134,8 +134,7 @@ binary-common: dh_installexamples dh_installdebconf dh_installlogrotate - dh_installinit -R -Nopenvswitch-switch - dh_installinit -R -popenvswitch-switch --error-handler=init_script_error + dh_installinit -R dh_installcron dh_installman --language=C dh_link -- 2.30.2