Initscript uses svc_(start|stop). These functions don't appear to be supported
Created attachment 273723 [details, diff]
Remove svc_(start|stop) and make openrc compatible
Just remove the custom restart() function as its not providing much value and openrc's default of calling stop() & start() is going to provide the same functionality.
Apart from their probably obsolete mention in a comment, how does that comment's presence do anything in openrc? How does removing the restart function help?
Dead code elimination, removal of a misleading function.... that's about it.
Does it help/hinder anything under openrc? Functionally, not really since openrc is going to ignore it anyway, but removing the appearance of functionality is probably a good thing. Before today, I assumed that openrc _didn't_ ignore restart() since several scripts still implemented it.
OK, that makes sense. Removed. Thanks for reporting.