tvrzna / emptty

Dead simple CLI Display Manager on TTY
MIT License
696 stars 26 forks source link

openrc service should set retry. #92

Closed amano-kenji closed 1 year ago

amano-kenji commented 1 year ago

/etc/init.d/transmission-daemon has

retry="TERM/45/QUIT/15"

/etc/init.d/fancontrol has

FANCONTROL_TERMTIMEOUT=${FANCONTROL_TERMTIMEOUT:-"TERM/60/KILL/5"}
retry="${FANCONTROL_TERMTIMEOUT}"

so that you can set retry in /etc/conf.d/fancontrol

Something like this should be in emptty openrc service so that ~/.config/emptty is given enough time to close extra programs.

tvrzna commented 1 year ago

What configuration would you suggest? I've tried restart of emptty service and it seems like that main interruption signal is TERM. TERM/5/KILL/5?

amano-kenji commented 1 year ago

Something like

TIMEOUT=${TIMEOUT:-"TERM/60/KILL/15"}
retry="${TIMEOUT}"

should be good because it gives emptty a plenty of time to gracefully shut down and allows users to configure retry. This is better than failing to stop.

tvrzna commented 1 year ago

I've tested with Alpine Linux and it seems to be good for handling issues. Thank you for the suggestion.