Pushing after merge considered harmful

Brian de Alwis briandealwis at gmail.com
Fri Jan 29 17:03:15 GMT 2010


On 29-Jan-2010, at 9:45 AM, Martin Pool wrote:
> 2010/1/29 Brian de Alwis <briandealwis at gmail.com>:
>> I just tried experimenting by setting 'append_revisions_only = true', but it appears 'true' must be 'True' with a capital 'T'.  Is there some reason why it doesn't do case-invariant checking?
> 
> No good reason that I know of; patches welcome.


Done: 
https://code.launchpad.net/~slyguy/bzr/bzr.strict_append_revisions_only/+merge/18267

I actually made the interpretation more strict: if set, it must be either 'true' or 'false'.  Given the consequences if mis-set, it seemed like a good place to err on the side of caution.

Brian.



--
On bike helmets: "If you think your hair is more important than your brain, you're probably right."  (B. J. Wawrykow)




More information about the bazaar mailing list