Subject: Re: [PATCHv5 1/6] Gitweb: add ignore and clean rules for minified files

From: Jakub Narebski <[email-masked]>
Date: Thu, 1 Apr 2010 10:24:48 +0200

On Thu, 1 April 2010, Mark Rada wrote:

> Signed-off-by: Mark Rada <[email-masked]>

For what it's worth:
Acked-by: Jakub Narebski <[email-masked]> 

> diff --git a/Makefile b/Makefile
> index 6d73a8c..5384d33 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2089,7 +2089,7 @@ clean:
>  	$(RM) $(htmldocs).tar.gz $(manpages).tar.gz
>  	$(MAKE) -C Documentation/ clean
>  ifndef NO_PERL
> -	$(RM) gitweb/gitweb.cgi
> +	$(RM) gitweb/gitweb.{cgi,min.*}
>  	$(MAKE) -C perl clean
>  endif

Minor nit: this would be first line that uses {a,b} shell glob 
construct.  Why not write it as:

  -	$(RM) gitweb/gitweb.cgi
  +	$(RM) gitweb/gitweb.cgi gitweb/gitweb.min.*

-- 
Jakub Narebski
Poland