On Fri, 20 Oct 2006 10:03:16 -0400
Aaron Bentley <aaron.bentley@utoronto.ca> wrote:
Perhaps I missed something in the earlier mails about this feature.
As I understood it, the email sent has a combined diff that shows
the net effect of all the commits included in the bundle. (Whereas
the current Cogito version only shows a diffstat)
If the recipient of such a bundle is unable to extract the diff of
each separate commit included in the bundle then I can't see any
value in the feature at all. But showing a combined diff in the
email may have marginal value, so long as when the bundle is
imported into the recipient repository the individual commits
are available.
A combined diff of a bunch of changes would usually be most _unwelcome_
for review on lkml. The constant refrain is to ask people to split their
changes up into smallish individual patches for review.
Sean
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html