threads / patch / 32009

patchRe: [PATCH] gitweb.perl: fix %highlight_ext mappings

Subject: Re: [PATCH] gitweb.perl: fix %highlight_ext mappings

## tl;dr

3 messages between Nov 2, 2012 and Nov 4, 2012. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

rh· Nov 2, 2012, 21:12 UTC · lore

[PATCH] gitweb.perl: fix %highlight_ext mappings

The previous change created a dictionary of one-to-one elements when the intent was to map mutliple related types to one main type. e.g. bash, ksh, zsh, sh all map to sh since they share similar syntax This makes the mapping as the original change intended.

Signed-off-by: rh <richard_hubbe11@lavabit.com>
Show changes to 2 files +9 −9

gitweb.cgi.orig, gitweb.cgi

diff --git a/gitweb.cgi.orig b/gitweb.cgi
index 060db27..155b238 100755
--- a/gitweb.cgi.orig
+++ b/gitweb.cgi
@@ -246,19 +246,19 @@ our %highlight_basename = (
 	'Makefile' => 'make',
 );
 # match by extension
+
 our %highlight_ext = (
 	# main extensions, defining name of syntax;
 	# see files in /usr/share/highlight/langDefs/ directory
-	map { $_ => $_ }
-		qw(py c cpp rb java css php sh pl js tex bib xml awk bat ini spec tcl sql make),
+	(map { $_ => $_ } qw(py rb java css js tex bib xml awk bat ini spec tcl sql)),
 	# alternate extensions, see /etc/highlight/filetypes.conf
-	'h' => 'c',
-	map { $_ => 'sh'  } qw(bash zsh ksh),
-	map { $_ => 'cpp' } qw(cxx c++ cc),
-	map { $_ => 'php' } qw(php3 php4 php5 phps),
-	map { $_ => 'pl'  } qw(perl pm), # perhaps also 'cgi'
-	map { $_ => 'make'} qw(mak mk),
-	map { $_ => 'xml' } qw(xhtml html htm),
+	(map { $_ => 'c'   } qw(c h)),
+	(map { $_ => 'sh'  } qw(sh bash zsh ksh)),
+	(map { $_ => 'cpp' } qw(cpp cxx c++ cc)),
+	(map { $_ => 'php' } qw(php php3 php4 php5 phps)),
+	(map { $_ => 'pl'  } qw(pl perl pm)), # perhaps also 'cgi'
+	(map { $_ => 'make'} qw(make mak mk)),
+	(map { $_ => 'xml' } qw(xml xhtml html htm)),
 );
 
 # You define site-wide feature defaults here; override them with
Jeff King· Nov 4, 2012, 13:48 UTC · re: rh · lore
On Fri, Nov 02, 2012 at 02:12:26PM -0700, rh wrote:
Show 11 quoted lines
> The previous change created a dictionary of one-to-one elements when
> the intent was to map mutliple related types to one main type.
> e.g. bash, ksh, zsh, sh all map to sh since they share similar syntax
> This makes the mapping as the original change intended.
> 
> Signed-off-by: rh <richard_hubbe11@lavabit.com>
> 
> diff --git a/gitweb.cgi.orig b/gitweb.cgi
> index 060db27..155b238 100755
> --- a/gitweb.cgi.orig
> +++ b/gitweb.cgi

Close on the format. There should be a "---" after the sign-off but before the diff. I can fix it up locally (and the patch looks good to me).

However, one final thing: the point of the sign-off is to indicate that you are legally OK to release the code under the DCO. For that reason, we usually require a real name (not rh). I can guess at your real name from your email, but I'd rather be sure. Can you provide it?

-Peff
rh· Nov 4, 2012, 17:44 UTC · re: Jeff King · lore

On Sun, 4 Nov 2012 08:48:41 -0500 Jeff King <peff@peff.net> wrote:

Show 22 quoted lines
> On Fri, Nov 02, 2012 at 02:12:26PM -0700, rh wrote:
> 
> > The previous change created a dictionary of one-to-one elements when
> > the intent was to map mutliple related types to one main type.
> > e.g. bash, ksh, zsh, sh all map to sh since they share similar
> > syntax This makes the mapping as the original change intended.
> > 
> > Signed-off-by: rh <richard_hubbe11@lavabit.com>
> > 
> > diff --git a/gitweb.cgi.orig b/gitweb.cgi
> > index 060db27..155b238 100755
> > --- a/gitweb.cgi.orig
> > +++ b/gitweb.cgi
> 
> Close on the format. There should be a "---" after the sign-off but
> before the diff. I can fix it up locally (and the patch looks good to
> me).
> 
> However, one final thing: the point of the sign-off is to indicate
> that you are legally OK to release the code under the DCO. For that
> reason, we usually require a real name (not rh). I can guess at your
> real name from your email, but I'd rather be sure. Can you provide it?

Roger wilco. Resubmitting. Thanks for the time and consideration.

> 
> -Peff

← back to recent threads