Re: [PATCH 1/4] instaweb: ignore GITWEB_CONFIG_SYSTEM
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:49:28
Jonathan Nieder [off-list ref] writes:
The configuration in /etc/gitweb.conf might specify any old layout; in particular, it is likely not to be identical to that which git instaweb sets up. Noticed by Uwe Kleine-König.
I am not sure if this is a reasonable change by reading the above. In some cases, the file may stale, but in other cases, it may specify what the site administrator wants its users to be using, no?
quoted hunk
Reported-by: Uwe Kleine-König <redacted> Cc: Eric Wong <redacted> Signed-off-by: Jonathan Nieder <redacted> --- git-instaweb.sh | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)diff --git a/git-instaweb.sh b/git-instaweb.sh index f608014..6279f4d 100755 --- a/git-instaweb.sh +++ b/git-instaweb.sh@@ -375,7 +375,8 @@ script=' s#^(my|our) \$projectroot =.*#$1 \$projectroot = "'$(dirname "$fqgitdir")'";#; s#(my|our) \$gitbin =.*#$1 \$gitbin = "'$GIT_EXEC_PATH'";#; s#(my|our) \$projects_list =.*#$1 \$projects_list = \$projectroot;#; -s#(my|our) \$git_temp =.*#$1 \$git_temp = "'$fqgitdir/gitweb/tmp'";#;' +s#(my|our) \$git_temp =.*#$1 \$git_temp = "'$fqgitdir/gitweb/tmp'";#; +s#(my|our) \$GITWEB_CONFIG_SYSTEM =.*#$1 \$GITWEB_CONFIG_SYSTEM = "";#;'
Perhaps moving the last ' to its own line would help maintaining the script in the longer term?
gitweb_cgi () {
cat > "$1.tmp" <<\EOFGITWEB
--
1.7.2.2