On 2014-10-14 Tue 00:24 AM |, Antoine Jacoutot wrote:
> 
> Makes sense yes. Not sure I'd want a function just for that one liner though.
> I'll commit something tomorrow.
> 

Nice one, using shell internals.

This restricts the listing to files which are also executable:


Index: rcctl.sh
===================================================================
RCS file: /cvs/src/usr.sbin/rcctl/rcctl.sh,v
retrieving revision 1.45
diff -u -p -r1.45 rcctl.sh
--- rcctl.sh    15 Oct 2014 07:38:24 -0000      1.45
+++ rcctl.sh    15 Oct 2014 09:01:35 -0000
@@ -39,10 +39,12 @@ needs_root()
 ls_rcscripts() {
        local _s
 
-       cd /etc/rc.d && set -- *
+       cd /etc/rc.d || exit
+       set -- *
        for _s; do
                [ "${_s}" = "rc.subr" ] && continue
-               [ ! -d "${_s}" ] && echo "${_s}"
+               [[ -d "${_s}" ]] && continue
+               [[ -f "${_s}" && -x "${_s}" ]] && echo "${_s}"
        done
 }
 

Reply via email to