diff options
author | Jakub Narebski <jnareb@gmail.com> | 2011-06-24 00:59:26 +0400 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2011-06-27 13:11:41 +0400 |
commit | c0175f92c7da1c379c25b5a70d53b9d550bcc11d (patch) | |
tree | b95e1be86954d1b5edc8e196d71e5f5a59b5475f | |
parent | 48bf76ca93c7569660712c12b3c4f86c60803352 (diff) |
git-instaweb: Move all actions at the end of script
As a nice side-effect now the order of parameters does not matter:
$ git instaweb --httpd=apache2 --start
is now (after this patch) the same as
$ git instaweb --start --httpd=apache2
Before this commit --start, --stop, --restart (and their subcommand
versions start, stop, restart) exited immediately.
This is preparatory work for making start/restart check that correct
configuration is set up; this change was required to have access in
start_httpd to requested web browser etc.
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Acked-by: Eric Wong <normalperson@yhbt.net>
-rwxr-xr-x | git-instaweb.sh | 27 |
1 files changed, 20 insertions, 7 deletions
diff --git a/git-instaweb.sh b/git-instaweb.sh index 2be22a0dfe..9a2f20f6ad 100755 --- a/git-instaweb.sh +++ b/git-instaweb.sh @@ -27,6 +27,7 @@ httpd="$(git config --get instaweb.httpd)" root="$(git config --get instaweb.gitwebdir)" port=$(git config --get instaweb.port) module_path="$(git config --get instaweb.modulepath)" +action="browse" conf="$GIT_DIR/gitweb/httpd.conf" @@ -148,17 +149,13 @@ while test $# != 0 do case "$1" in --stop|stop) - stop_httpd - exit 0 + action="stop" ;; --start|start) - start_httpd - exit 0 + action="start" ;; --restart|restart) - stop_httpd - start_httpd - exit 0 + action="restart" ;; -l|--local) local=true @@ -611,6 +608,22 @@ configure_httpd() { esac } +case "$action" in +stop) + stop_httpd + exit 0 + ;; +start) + start_httpd + exit 0 + ;; +restart) + stop_httpd + start_httpd + exit 0 + ;; +esac + gitweb_conf resolve_full_httpd |