Re: [PATCH] Re:gitweb: add readme to overview page

2012-11-12 Thread Junio C Hamano
Heinrich Schuchardt  writes:

> In this version of the patch the formatting has been corrected.
>
> Warnings for double / in filenames are avoided.
>
> Signed-off-by: Heinrich Schuchardt 
>
> ---

The above is *not* a proper log message.

Those of us who are reading the messages on the list would know that
this is a replacement for your earlier

Message-ID: <1352647962-21910-1-git-send-email-xypron.g...@gmx.de>

where it has more proper description of the change, but because
nobody will be applying that earlier one to the history (instead,
you would want this version to be applied), the description is lost
in the history.

>  gitweb/gitweb.perl |   12 
>  1 file changed, 12 insertions(+)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 10ed9e5..699ffac 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -6369,6 +6369,18 @@ sub git_project_list {
>   }
>  
>   git_project_search_form($searchtext, $search_use_regexp);
> + # If XSS prevention is on, we don't include README.html.
> + # TODO: Allow a readme in some safe format.
> + my $path = "";
> + if (defined $project_filter) {
> + $path = "/$project_filter";
> + }
> + if (!$prevent_xss && -s "$projectroot$path/README.html") {
> + print "readme\n" .
> + "\n";
> + insert_file("$projectroot$path/README.html");
> + print "\n\n"; # class="readme"
> + }
>   git_project_list_body(\@list, $order);
>   git_footer_html();
>  }
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] Re:gitweb: add readme to overview page

2012-11-11 Thread Heinrich Schuchardt
In this version of the patch the formatting has been corrected.

Warnings for double / in filenames are avoided.

Signed-off-by: Heinrich Schuchardt 

---
 gitweb/gitweb.perl |   12 
 1 file changed, 12 insertions(+)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 10ed9e5..699ffac 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -6369,6 +6369,18 @@ sub git_project_list {
}
 
git_project_search_form($searchtext, $search_use_regexp);
+   # If XSS prevention is on, we don't include README.html.
+   # TODO: Allow a readme in some safe format.
+   my $path = "";
+   if (defined $project_filter) {
+   $path = "/$project_filter";
+   }
+   if (!$prevent_xss && -s "$projectroot$path/README.html") {
+   print "readme\n" .
+   "\n";
+   insert_file("$projectroot$path/README.html");
+   print "\n\n"; # class="readme"
+   }
git_project_list_body(\@list, $order);
git_footer_html();
 }
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html