[Bug 193728] New: [patch] net/serviio - add rc script exit codes for service monitoring daemons

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193728

            Bug ID: 193728
           Summary: [patch] net/serviio - add rc script exit codes for
                    service monitoring daemons
           Product: Ports Tree
           Version: Latest
          Hardware: Any
                OS: Any
            Status: Needs Triage
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: netchild at FreeBSD.org
          Reporter: jason.unovitch at gmail.com
          Assignee: netchild at FreeBSD.org
             Flags: maintainer-feedback?(netchild at FreeBSD.org)

Created attachment 147420
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=147420&action=edit
patch adding exit codes to the serviio rc script status function

Serviio's rc script currently always exits with a 0 from the echo lines in its
status function.  This confuses service monitoring daemons as it looks like
Serviio is actually running.  A patch that has explicit exit codes is attached
that should improve the behavior with anything that expects a 0 as meaning the
service is working.


##### BEFORE
root at test-serviio:/usr/local/etc/rc.d # service serviio onestatus
serviio is not running
root at test-serviio:/usr/local/etc/rc.d # echo $?
0
root at test-serviio:/usr/local/etc/rc.d # service serviio onestatus
serviio is running as pid 40239
root at test-serviio:/usr/local/etc/rc.d # echo $?
0

##### AFTER
root at test-serviio:/usr/local/etc/rc.d # service serviio onestatus
serviio is running as pid 40743
root at test-serviio:/usr/local/etc/rc.d # echo $?
0
root at test-serviio:/usr/local/etc/rc.d # service serviio onestatus
serviio is not running
root at test-serviio:/usr/local/etc/rc.d # echo $?
1

--- Comment #1 from Bugzilla Automation <bugzilla at FreeBSD.org> ---
Auto-assigned to maintainer netchild at FreeBSD.org

-- 
You are receiving this mail because:
You are the assignee for the bug.