[MERGE REVIEW] Tweaks to bundle merging
Aaron Bentley
aaron.bentley at utoronto.ca
Sat Jun 17 15:50:21 BST 2006
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Michael Ellerman wrote:
> On 6/16/06, Aaron Bentley <aaron.bentley at utoronto.ca> wrote:
>> It doesn't have to be a roll-up, though-- it could be just like the
>> other diffs. We made it a roll-up so that you could review the combined
>> changes using it.
>
> Yeah, that's what I meant, show each diff individually. That has the
> other nice property that the messages correspond to the diff hunks.
Personally, I prefer to see all the changes at once. I don't want to
see all the false starts and reverted changes. What I want to see is
what the bundle will do to a tree if I merge it.
> I think this would work even better if all the trailing blocks
> (revision_id/sha1/etc) could drop to the bottom. And maybe be
> base64'ed there too.
I do find it nice to see all the commit messages (contradicting my first
point a bit), but the bottoms mostly aren't useful, I agree. Except
that I can guess whether the bundle will apply by looking at its base
revision.
>> Plus it means that we can check the target revision's signature without
>> having to install anything.
>
> Ok, that's nice I guess. But not sure if it justifies the slight
> strangness of the current format.
It wasn't planned, but I think it's very useful for submit-by-email and
things. The main reason was because the roll-up's more interesting than
the series of patches that produce it.
The format is flexible, though. A bundle that didn't have any roll-ups
would still be valid.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFElBaJ0F+nu1YWqI0RAubRAJ9z6YCJW0iMGM8UdgvBg7jpQe5bHgCfUJGe
K2jkZDRddQPwm6UPTDfI5fM=
=m4uh
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list