[MERGE] HACKING transformed into Bazaar Developer Guide

Ian Clatworthy ian.clatworthy at internode.on.net
Wed May 2 05:54:37 BST 2007


Aaron Bentley wrote:
> Some of this TDD stuff is about testing in general, rather than Test
> Driven Development.  While we can recommend TDD, what we require is
> proper testing.  So I think we need a different headline.
>
>   
Fixed as discussed on IRC.
> SIGQUIT is not SIGINT; SIGINT is caused by CTRL-C.  CTRL-\ is required
> to produce SIGQUIT, not CTRL-C.
>
>   
I think this was mainly a formatting issue with a \ at the end of line 
confusing the output. Corrected.
> There is no such thing as a bzr changeset.
>
> Also, I recommend using merge directives rather than sending a diff+branch.
>   
>
> Please do *NOT* put [PATCH] or [MERGE] in the subject line if you don't
> want it to be merged.
>
>   
Section improved to address all of these.
>> +If you read a patch please reply and say so.  We can use a numeric scale
>> +of -1, -0, +0, +1, meaning respectively "really don't want it in current
>> +form", "somewhat uncomfortable", "ok with me", and "please put it in".
>> +Anyone can "vote".   (It's not really voting, just a terse expression.)
>> +
>> +If something gets say two +1 votes from core reviewers, and no
>> +vetos, then it's OK to come in.  Any of the core developers can bring it
>> +into their integration branch, which I'll merge regularly.  (If you do
>> +so, please reply and say so.)
>>     
>
> ^^^^ This is out of date, now that BB has formalized voting.
>
>   
Using the BB help as I guide, I've updated this text. I had to clarify 
the meanings a bit though because 0+ and 1+ don't really translate to 
"ok by me" and "put it in" - not everyone has access to do the latter 
and I guess that was implied in earlier days.

Latest bundle attached. It includes the fix Alexander requested as well. 
Once again, the net HTML result is attached for those without rst2html 
installed.

Ian C.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: HACKING-as-developer-guide3.patch
Type: text/x-patch
Size: 107405 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070502/e6789197/attachment-0001.bin 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20070502/e6789197/attachment-0001.html 


More information about the bazaar mailing list