Skip to content

rc: service.c: adds status, restart, list, exit code and errors reporting#241

Open
wellloaded wants to merge 1 commit intoFreshTomato-Project:arm-masterfrom
wellloaded:patch-1
Open

rc: service.c: adds status, restart, list, exit code and errors reporting#241
wellloaded wants to merge 1 commit intoFreshTomato-Project:arm-masterfrom
wellloaded:patch-1

Conversation

@wellloaded
Copy link
Contributor

root@sparrow:/mnt/USB# service help
Usage: service start|stop|restart|status
service list

start Start the service
stop Stop the service
restart Stop then start the service
status Show whether the service process is running

Exit status: 0 on success, 1 on failure.
Run 'service list' for all available service names.

##########################################

  • adds "help"
  • adds "list" of services
  • adds "restart" as an action, reports on pid where relevant
  • adds "status" to report on services
  • prints an error (instead than the hardcoded Done.) for issues e.g. service didn't restart
  • reports error on wrong service names
  • reports error on wrong action names
  • handles exit codes properly

This change is an enabler for a new "Status / Services <-> status-services.asp" page on the GUI

root@sparrow:/mnt/USB# service help
Usage:  service <name> start|stop|restart|status
        service list

  start    Start the service
  stop     Stop the service
  restart  Stop then start the service
  status   Show whether the service process is running

Exit status: 0 on success, 1 on failure.
Run 'service list' for all available service names.

##########################################
- adds "help"
- adds "list" of services
- adds "restart" as an action, reports on pid where relevant
- adds "status" to report on services
- prints an error (instead than the hardcoded Done.) for issues e.g. service didn't restart
- reports error on wrong service names
- reports error on wrong action names
- handles exit codes properly

This change is an enabler for a new "Status / Services <-> status-services.asp" page on the GUI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant