Jakub Narebski <jnareb@xxxxxxxxx> writes:
>> 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.*
Very good point and it is not a "minor nit"; I cannot apply the patch
without your fix.
My understanding is that you Ack 1 (with this fix), 2 (with another
paragraph in the log message), and 5. Eric Acked 4 and 6 needs to be
reworked. I wonder if anybody has comments on 3; I didn't look very
closely at it, but it looked sane to me.
With that understanding, I'll queue the whole thing to 'pu', so that they
can be reworked and replaced before hitting 'next'.
Thanks, all.
--
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