Comment 4 for bug 485625

Revision history for this message
Aaron Bentley (abentley) wrote : Re: [Bug 485625] Re: mail out interdiffs when updating review diffs

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Robert Collins wrote:
> On Fri, 2009-11-20 at 14:21 +0000, Aaron Bentley wrote:
>> This could easily be a cause of unwanted email, if every time the branch
>> is pushed to, new interdiffs are emailed.
>
> John suggests having a button to trigger this.
>
> I think that the resubmit button could be repurposed, as at the moment
> losing the conversation history is a misfeature for most folk I know.

We plan to show the conversation from previous merge proposals. I don't
think changing the meaning of resubmit would work.

> I will note though, that you don't have to push on every commit :).
>
> Oh, and the moving diff should be the diff between the previews

You mean a diff of the past and present preview diffs?

, not the
> changes along the branch - so merges from trunk and other 'things are
> not changed' pushes /will not/ trigger mail anyway, as the preview won't
> change.

I don't follow. Diffs include line numbers and context-- these may both
change when the target branch is merged into the source branch.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAksG+j0ACgkQ0F+nu1YWqI2JUACfUDewkv77lZTUrM3mO6ZQivVn
U04An2oDBHqMNzqo1InFjKhdD7nJlob2
=fy+f
-----END PGP SIGNATURE-----