Package: xprint
Version: 1:1.0.2-1
Severity: normal
Tags: patch

Hi.

As LSB compliance is at least a pet release goal for etch, I file this 
as a non-wishlist bug. According to LSB 3, "force-reload" should only 
restart processes if the process is already running and doesn't support
"reload". Furthermore, it doesn't make sense to show "reload" as a
option to the init script in the Usage output. I therefor suggest applying
the attached patch to fix this issue.

Regards,
Sven


-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (90, 'testing'), (50, 'unstable')
Architecture: amd64 (x86_64)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.8-12-amd64-generic
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
--- debian/xprint-common.xprint.init.orig       2006-07-10 17:53:06.403547580 
+0200
+++ debian/xprint-common.xprint.init    2006-07-10 18:14:29.316047161 +0200
@@ -36,8 +36,8 @@
 # 3. Query the list of available printers.
 #
 # Additional tasks are:
-# 4. Restart ('restart'|'force-reload') and conditional restart
-#    ('condrestart'/'try-restart') for Linux support
+# 4. Restart ('restart') and conditional restart
+#    ('condrestart'|'try-restart'|'force-reload') for Linux support
 # 5. Wrapping of application call with setting XPSERVERLIST ('wrapper')
 #
 # Usage:
@@ -1171,7 +1171,7 @@
         ;;
 
     ## Restart Xprint servers
-    'restart'|'force-reload')
+    'restart')
         do_restart
         ;;
 
@@ -1183,7 +1183,7 @@
         ;;
 
     ## Restart Xprint only if it is already running
-    'condrestart'|'try-restart')
+    'condrestart'|'force-reload'|'try-restart')
         # only restart if it is already running
         [ -f /var/lock/subsys/xprint ] && do_restart || :
         ;;
@@ -1238,7 +1238,7 @@
     
     ## Print usage
     *)
-        msg "Usage: $0 { start | stop | restart | reload | force-reload | 
status | condrestart | try-restart | wrapper | lsprinters | get_xpserverlist | 
diag }"
+        msg "Usage: $0 { start | stop | restart | force-reload | status | 
condrestart | try-restart | wrapper | lsprinters | get_xpserverlist | diag }"
         exit 2
 esac
 exit 0

Reply via email to