Bug#728682: Patch

2016-12-20 Thread Felipe Sateler
Control: tags -1 pending

On Sat, 10 Dec 2016 21:36:33 +0100 Christian Hofstaedtler
 wrote:
> Control: tags -1 + patch
>
> Proposed patch attached.

Thanks. I have applied the patch. I also find "just in case warnings"
quite useless.

Saludos,



Bug#728682: Patch

2016-12-10 Thread Christian Hofstaedtler
Control: tags -1 + patch

Proposed patch attached.

-- 
 ,''`.  Christian Hofstaedtler 
: :' :  Debian Developer
`. `'   7D1A CFFA D9E0 806C 9C4C  D392 5C13 D6DB 9305 2E03
  `-

>From ceae57b4ae857680c2526d2e7c3a149ae5c23dec Mon Sep 17 00:00:00 2001
From: Christian Hofstaedtler 
Date: Sat, 10 Dec 2016 20:34:05 +
Subject: [PATCH] Align policy-rc.d existing/nonexisting case for custom
 actions

Previously, when calling invoke-rc.d with a "custom" action, it
would print an error when a policy-rc.d file was installed, but
it would be silent if no such file was installed.

Closes: #728682
---
 script/invoke-rc.d | 11 ---
 1 file changed, 11 deletions(-)

diff --git a/script/invoke-rc.d b/script/invoke-rc.d
index ed9028d..7d839bf 100755
--- a/script/invoke-rc.d
+++ b/script/invoke-rc.d
@@ -257,17 +257,6 @@ fi
 #NOTE: It may not be obvious, but "$@" from this point on must expand
 #to the extra initscript parameters, except inside functions.
 
-## sanity checks and just-in-case warnings.
-case ${ACTION} in
-start|stop|force-stop|restart|reload|force-reload|status)
-   ;;
-*)
-   if test "x${POLICYHELPER}" != x && test -x "${POLICYHELPER}" ; then
-   printerror action ${ACTION} is unknown, but proceeding anyway.
-   fi
-   ;;
-esac
-
 # Operate against system upstart, not session
 unset UPSTART_SESSION
 # If we're running on upstart and there's an upstart job of this name, do
-- 
2.11.0