Re: [ANNOUNCE] Example Cogito Addon - cogito-bundle
From: Aaron Bentley <hidden>
Date: 2016-06-15 22:42:44
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jakub Narebski wrote:
Aaron Bentley wrote:quoted
In Bazaar bundles, the text of the diff is an integral part of the data. It is used to generate the text of all the files in the revision.I thought that the diff was combined diff of changes.
It is. It's a description of how to produce revision X given revision Y, where Y is the last-merged mainline revision.
quoted
Bazaar bundles were designed to be used on mailing lists. So you can review the changes from the diff, comment on them, and if it seems suitable, merge them.If you have only mega-diff, you can comment only on this mega-diff.
That is what we prefer to review.
quoted
quoted
Although that might just make the email bigger for not a lot of gain.It's my understanding that most changes discussed on lkml are provided as a series of patches. Bazaar bundles are intended as a direct replacement for patches in that use case.As _series_ of patches. You have git-format-patch + git-send-email to format and send them, git-am to apply them (as patches, not as branch).
If you want to do it exactly the same way, you send a series of bundles. The bundle format can also support sending a single bundles that displays the series of patches, though there's currently no UI to select this.
I was under an impression that user sees only mega-patch of all the revisions in bundle together, and rest is for machine consumption only.
All of it is for machine consumption. The MIME-encoded sections are a series of patches. They're usually MIME-encoded to avoid confusion with the overview patch, but this is optional. I've attached an example of what a combined patch-by-patch bundle looks like. Aaron -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.1 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org iD8DBQFFOOyB0F+nu1YWqI0RAtU6AKCJndTNlTTPNnzxZX53lkBUUHTYkwCfePlG 7x3cjpYwh8LXEb5ZWXXmu6s= =6Lgv -----END PGP SIGNATURE-----
Attachments
- hello-world.patch [text/x-patch] 1808 bytes · preview