Code

git-instaweb: Check that correct config file exists for (re)start
authorJakub Narebski <jnareb@gmail.com>
Thu, 23 Jun 2011 21:01:03 +0000 (23:01 +0200)
committerEric Wong <normalperson@yhbt.net>
Mon, 27 Jun 2011 09:11:41 +0000 (09:11 +0000)
commit5ad6d387f19d9dd28b323f5cfea8663cb7b74617
treed8833346f606870b6d319e9e34c0cd5b723f85ff
parentc0175f92c7da1c379c25b5a70d53b9d550bcc11d
git-instaweb: Check that correct config file exists for (re)start

Currently start/restart does not generate any configuration files for
spawning a new instance.  This means that

  $ git instaweb --http=<server> --start

might pick up stale 'httpd.conf' file for a different web server
(e.g. for default lighttpd when requesting apache2).

This commit changes that, and makes git-instaweb generate web server
config file and/or gitweb config file if don't exists.

This required naming config files after the name of web server
(alternate solution would be to somehow mark for which web server was
config file generated).

Note that web servers that embed configuration in server script file,
namely webrick and plackup, and which delete "$conf" in their *_conf
function, would have their config (server script) always regenerated.

Note: this commit introduces a bit of code repetition (but only a few
lines).

Reported-by: Gurjeet Singh <singh.gurjeet@gmail.com>
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Acked-by: Eric Wong <normalperson@yhbt.net>
Documentation/git-instaweb.txt
git-instaweb.sh