Added section "Command format"
|Deletions are marked like this.||Additions are marked like this.|
|Line 115:||Line 115:|
|1. Merge changeset code into core||1. Merge changeset code into core (./)|
|Line 129:||Line 129:|
|=== Command format ===
Command format will look like this:
usage: bzr submit [BRANCH]
--dry-run Print message instead of sending
--help, -h show help message
--message ARG, -m
--subject ARG Message subject
If no subject is given the last log message is used and prefixed with [PATCH].
If no message is given an editor is launched.
|Line 135:||Line 150:|
|I (HermannKraus) am doing this job as my Summer of Code project. My code will be based on Aaron Bentleys changeset code which already provides a good format for transmitting all the meta data. lete.||I (HermannKraus) am doing this job as my Summer of Code project. My code will be based on Aaron Bentleys changeset code which already provides a good format for transmitting all the meta data.|
|Line 152:||Line 167:|
|* HermannKraus is currently working on automatic gpg signing and verification.|| * After long discussions we came to the conclusion that signing should be left to the mail program and not be done in bazaar.
* Currently I'm working on a plugin that provides the submit command.
Launchpad Entry: https://launchpad.net/products/bzr/+spec/SubmitByMail
PQM provides a way to submit merge requests by email and have them automatically applied. It might be nice if this were smoothly integrated with bzr.
Erik Bågfors <firstname.lastname@example.org> to mbp, Bazaar-NG How about adding this kind of functionality into bzr itself? Look at how darcs handles it. It's one if the nicest things about darcs. a darcs "server" can simply be a mail address you mail to, that runs all mails trough pgp and then applies them if they match (you can add tests as well by just adding "apply test" to the repo config file). for example a procmail can look like this :0: * ^TOmy darcs repo |(umask 022; darcs apply --reply email@example.com \ --repodir /path/to/your/repo --verify /path/to/the/allowed_keys) That's a dead simple way to set up a "server" for trusted people to submit to. I don't know how pqm works and it might be lot's better, but having this functionality built into bzr has lot's of advantages I think.
This has the big advantage that no special kind of server is required. A procmail rule is something almost anyone can set up, including on a desktop/firewalled/not-permanently-connected machine.
It would be nice to send the changes as a series of bzr changesets (as mentioned on BzrWishlist).
There should be one command to send the changeset (bzr submit), and one to apply the changeset (bzr apply), to be called manually or from a procmail rule.
There are lots of tools to do some sort of diff, mail it, and then apply it on the other end. A simple bzr diff -r -10 | gpg --cl | mail firstname.lastname@example.org will be able to do a reasonable job of sending a signed change to a mailing list. It is assumed that people who want a bzr submit command would also like to attach some meta-data. It is possible to create a text-based changeset format, which contains all of the meta-information which bzr maintains, and apply it on the other end, so that it has all the features of doing a bzr merge against the original branch.
User grabs a public branch, finds a bug, and wants to submit back a small patch back to the mailinglist. The nice use case would be:
$ bzr branch http://public.com/project $ cd project <hack hack> $ bzr commit -m "Fixed my little bug" $ bzr submit
People can review the final changes in their mail program, and finally decide to apply this change and commit it:
<save the email to a file> cat file | bzr apply # or bzr apply file <double check things are okay> bzr commit -m "Applied the change from foo"
As a maintainer, I use bzr submit to submit my changes to a pqm. However, since others would not have commit access to the pqm, I want to specify that submissions should go to the mailing list. I publish my branch using rsync.
- I would like to automatically maintain a branch, such that I can submit to it using email, with a simple procmail filter.
To allow bzr branch to know how to set the submit target, there should be a file in the .bzr/ directory indicating the target. To allow for a different target for people who branch from me, versus my default branch target, there should be 2 files. (.bzr/submit-to and .bzr/children-submit-to) bzr branch should be smart, such that when it creates a branch, it will use .bzr/children-submit-to if it is available.
There would be one command, bzr submit, with several options and configuration variables.
Basic usage would be:
bzr submit -r ancestor:http://target/uri.. --output /tmp/changeset.bzr
bzr submit -r ancestor:http://target/uri.. --mail email@example.com
Of course -r would be standard revision specification. It would be exclusive for the start (except when only one revision is given) for convenience of the above notation.
There would be a shortcut
bzr submit --target http://target/uri ...
for the -r option. This value would be remembered by config and default to parent or push target.
There would be a bit of DWIMmeryFootNote(DWIM = Do What I Mean) for the second argument -- if a non-option argument is given
and looks like an email address (can be forced with mailto:), it is treated as argument to --mail.
if the argument looked like URL or path, which exists and is a directory, it would be treated as argument of --target.
else it would be treated as argument to --output (- should be understood to mean stdout)
-- Be carefull that this is ambiguous. How do you save in a file containing a '@' in its name? To you distinguish between a filename (--output) and a path (--target), you need to see if the branch exists. If you misstyped the name, you'll get one behavior instead of the other. I find it confusing (but if the non-ambiguous options are there, that's probobly OK) --MatthieuMoy
Plus of course this would be remembered by config as well. So if you once did:
bzr submit http://bazaar-ng.org/bzr/bzr.dev firstname.lastname@example.org
you could then just do:
And it would just do the right thing.
There would be one more option, which would only be used together with --mail -- --mua. It would be a command to run the mailer. It could either be a format string (containing %(to)s for recipient address and %(file)s for the changeset file -- it would be assumed that the mua will make it an attachment). Plus for common MUAs (mailx, mutt, mh, af, gnus...) bzr could know how to pass the arguments, so one could only type in their name. There would of course be respective config option.
Merge changeset code into core
- Track default submit location
Is it useful to keep track of what has already been sent to a particular destination?
Command format will look like this:
usage: bzr submit [BRANCH] options: --dry-run Print message instead of sending --help, -h show help message --message ARG, -m --subject ARG Message subject
If no subject is given the last log message is used and prefixed with [PATCH]. If no message is given an editor is launched.
Questions and Answers
Summer of Code
I (HermannKraus) am doing this job as my Summer of Code project. My code will be based on Aaron Bentleys changeset code which already provides a good format for transmitting all the meta data.
- Automated signing of the patch with gpg.
- Samples rules for automatically applying the changes after verifying the senders identity.
- Remembering the address patches are submitted to and a default address that is active after branching.
- Configurable mailer command.
- Display a message before submitting a patch to a public mailing list.
RobHolland considers the following as the criteria for success of the Summer of Code project
'bzr branch http://bazaar-vcs.org/bzr/bzr.dev; <hack hack>; bzr commit; bzr submit' will email a signed bundle to the bzr mailing list
- 'bzr apply' is capable of applying a valid bundle
- 'bzr apply' gracefully handles a corrupted bundle
- 'bzr apply' can be set to automatically apply bundles from a set of gpg keys
- 'bzr apply' can be set to automatically apply any bundle with a valid signature
- 'bzr apply' can be set to automatically reject any bundle with an invalid signature
AaronBentley has a branch with support for changesets (now called bundles) at http://code.aaronbentley.com/bzr/bzrrepo/others/w-changeset/
- After long discussions we came to the conclusion that signing should be left to the mail program and not be done in bazaar.
- Currently I'm working on a plugin that provides the submit command.