From a85c0de1f87f1c3450ef8c4a341b050d07356368 Mon Sep 17 00:00:00 2001 From: schwarze Date: Sun, 24 Aug 2014 18:10:26 +0000 Subject: [PATCH] Start all error messages with "rcctl: " such that it is clear where they come from. While here, there is no need to check the argument of needs_root() before displaying it, it is always provided. ok ajacoutot@ jasper@ --- usr.sbin/rcctl/rcctl.sh | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/usr.sbin/rcctl/rcctl.sh b/usr.sbin/rcctl/rcctl.sh index cd172d6083f..19e3301fc8f 100644 --- a/usr.sbin/rcctl/rcctl.sh +++ b/usr.sbin/rcctl/rcctl.sh @@ -1,6 +1,6 @@ #!/bin/sh # -# $OpenBSD: rcctl.sh,v 1.17 2014/08/24 17:52:38 ajacoutot Exp $ +# $OpenBSD: rcctl.sh,v 1.18 2014/08/24 18:10:26 schwarze Exp $ # # Copyright (c) 2014 Antoine Jacoutot # @@ -29,7 +29,7 @@ usage() needs_root() { if [ "$(id -u)" -ne 0 ]; then - _rc_err "${0##*/} ${1:+$1: }need root privileges" + _rc_err "${0##*/} $1: need root privileges" fi } @@ -292,7 +292,7 @@ flags=$* if [ -n "$svc" ]; then if ! svc_is_avail $svc; then - _rc_err "service $svc does not exist" + _rc_err "${0##*/}: service $svc does not exist" fi elif [ "$action" != "status" ]; then usage @@ -301,10 +301,10 @@ fi if [ -n "$flag" ]; then if [ "$flag" = "flags" ]; then if [ "$action" != "enable" ]; then - _rc_err "\"flags\" can only be set with \"enable\"" + _rc_err "${0##*/}: \"flags\" can only be set with \"enable\"" fi if svc_is_special $svc; then - _rc_err "\"$svc\" is a special variable, cannot set \"flags\"" + _rc_err "${0##*/}: \"$svc\" is a special variable, cannot set \"flags\"" fi else usage @@ -331,7 +331,7 @@ case $action in ;; start|stop|restart|reload|check) if svc_is_special $svc; then - _rc_err "\"$svc\" is a special variable, no rc.d(8) script" + _rc_err "${0##*/}: \"$svc\" is a special variable, no rc.d(8) script" fi /etc/rc.d/$svc ${_RC_DEBUG} ${_RC_FORCE} $action ;; -- 2.20.1