Stephen Boyd <bebarino@xxxxxxxxx> writes: > Currently when format-patch is used with --attach or --inline the patch > attachment has the SHA1 of the commit for its filename. This replaces > the SHA1 with the filename used by format-patch when outputting to files. > > Fix tests relying on the SHA1 output and add a test showing how the > --suffix option affects the attachment filename output. > > Signed-off-by: Stephen Boyd <bebarino@xxxxxxxxx> > --- > Is modifying the rev_info struct correct? Most of the rev_info is about the overall traversal not about individual commits, and I think the new field you are adding is about a single commit you will update every time you switch to a new commit to process, so in that sense it may violate the general idea of what rev-info is, but I do not think it matters too much. -- 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