Petter Reinholdtsen <p...@hungry.com> writes: > Renaming the function will break anyone doing the 'alias' trick, so any > changes should be done in a backwards compatible way, I believe. > > Patches welcome. :)
The approach I was suggesting would look like this:
+ Enable this using
+ RELOAD_SIGNAL=1 # or other signal number
+ alias do_reload=do_reload_signal
+do_reload_signal() {
+ log_daemon_msg "Reloading $DESC configuration files" "$NAME"
+ start-stop-daemon --oknodo --stop --signal $RELOAD_SIGNAL \
+ --quiet --pidfile "$PIDFILE" --exec "$DAEMON"
+ log_end_msg $?
+}
(There could even be a default value for RELOAD_SIGNAL, if desired.)
Once that has landed, I was suggesting sending patches to everyone using
do_reload_sigusr1() right now that look like this:
-alias do_reload=do_reload_sigusr1
+SIGNAL=1
+alias do_reload=do_reload_signal
At which point do_reload_sigusr1 can be safely removed since no one is
using it.
signature.asc
Description: PGP signature
