Jakub Narebski <jnareb@xxxxxxxxx> writes: > Jakub Narebski wrote: > >> Shortlog: >> [PATCH 1/7] gitweb: Make pickaxe search a feature >> [PATCH 2/7] gitweb: Paginate history output >> [PATCH 3/7] gitweb: Use @hist_opts as git-rev-list parameters >> in git_history >> [PATCH 4/7] gitweb: Add parse_rev_list for later use >> [PATCH 5/7] gitweb: Use parse_rev_list in git_shortlog and git_history >> [PATCH 6/7] gitweb: Assume parsed revision list in git_shortlog_body >> and git_history_body >> [PATCH 7/7] gitweb: Set page to 0 if it is not defined, in git_history > > Should I resend corrected patch 1 (Make pickaxe search a feature; but > default to on), and patches 2 and 7 as one patch (Paginate history output)? I looked at and understood 1 and 2+7, although I cannot claim I looked at 2+7 deeply enough; cursory look told me they should be Ok. So is 3, but I suspect gitweb without --full-history might surprise some users. I do not know about 4, 5 and 6. I didn't look at them at all the first time you sent them out, since I got an impression that you did not understand how git-rev-list was supposed to work when you did them. Now Linus explained it to you, I suspect they would probably need to be rethought? - 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