Wow, I honestly didn't expect this idea to be so successful. I thought I was the only one using gitweb to send patches around, honestly 8-D On Sat, Nov 29, 2008 at 5:10 PM, Jakub Narebski <jnareb@xxxxxxxxx> wrote: > On Sat, 29 Nov 2008, Sverre Rabbelier wrote: >> >> If this does what I think it does I would be very happy with this >> feature :). Only yesterday I wanted to link someone to a patch I put >> up on repo.or.cz, but instead ended up telling them to download the >> snapshot. > > True. 'commitdiff_plain' wasn't good enough; what's more it suffers > from the same ambiguity as 'commitdiff', i.e. it is both means to > show diff _for_ a commit (perhaps selecting one of parents), and > showing diff _between_ two commits; the new 'patch' always shows > diff for a commit, or for a series of commits. Maybe commitdiff should me renamed to just be 'diff'. Also, I was in doubt about the name for the new view, and I did consider 'patchset' (which you mention in your email). I chose to stick with the shorter form in the end, since many people complain that gitweb already produces paths that are too long. -- Giuseppe "Oblomov" Bilotta -- 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