At Tuesday 29 January 2008 around 04:14:00 Yasushi SHOJI wrote: > commitdiff with raw, or plain format if you are reading the code, > doesn't convert any word from perl internal to utf8, which is set to > charset in http header. this cause a problem when commit includes non > ascii code. > > here is a few example in the git tree: > > http://git.kernel.org/?p=git/git.git;a=commitdiff_plain;h=6ba78238a82428281 >6944550edc4297dd2808a72 > http://git.kernel.org/?p=git/git.git;a=commitdiff_plain;h=e360bebf713b6b037 >68c62de8b94ddf9350b0953 > http://git.kernel.org/?p=git/git.git;a=commitdiff_plain;h=9459aa77a032621a2 >9d53605542844641cca843a > > This patch effectively revert the commitdiff plain part of the commit > > 59b9f61a3f76762dc975e99cc05335a3b97ad1f9 > > which converted from print to here-doc. but it doesn't > explain why in the commit log. > --- > gitweb/gitweb.perl | 12 ++++++------ > 1 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl > index 6256641..5d9ac1d 100755 > --- a/gitweb/gitweb.perl > +++ b/gitweb/gitweb.perl > @@ -5048,16 +5048,16 @@ sub git_commitdiff { > -expires => $expires, > -content_disposition => 'inline; filename="' . "$filename" . '"'); > my %ad = parse_date($co{'author_epoch'}, $co{'author_tz'}); > - print <<TEXT; > -From: $co{'author'} > -Date: $ad{'rfc2822'} ($ad{'tz_local'}) > -Subject: $co{'title'} > -TEXT > + print "From: " . to_utf8($co{'author'}) . "\n"; > + print "Date: " . to_utf8($ad{'rfc2822'}) . " " > + . to_utf8($ad{'tz_local'}) . "\n"; > + print "Subject: " . to_utf8($co{'title'}) . "\n"; > + > print "X-Git-Tag: $tagname\n" if $tagname; > print "X-Git-Url: " . $cgi->self_url() . "\n\n"; > > foreach my $line (@{$co{'comment'}}) { > - print "$line\n"; > + print to_utf8($line) . "\n"; > } > print "---\n\n"; > } Nice find, looks correct. You could print "Date" on one line instead of two, but FWIW Acked-by: İsmail Dönmez <ismail@xxxxxxxxxxxxx> -- Never learn by your mistakes, if you do you may never dare to try again. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
