[MERGE] Add a way to specify a template commit message
James Westby
jw+debian at jameswestby.net
Sun Feb 4 12:38:20 GMT 2007
On (04/02/07 07:29), Robert Collins wrote:
> Robert Collins has voted +1 (conditional).
> Status is now: Semi-approved
> Comment:
> The check for 'start_message != ""' seems wrong to me - an empty start
> message is still a start message.
I was just following the lead of the template message code. That
excludes an empty template message. I can see that that might be
different though, you can save on the separator adding removing if it is
empty. I'll make the change as you add.
> Just checking for None appears
> sufficient to me. Other than that, the test for using start_message
> should be a separate test function, not an extension of the current one
> - its a new test, and the mutter() call seems entirely spurious to me.
>
Again I was just following what was there. I was trying to be cautious
to avoid problems on things like win32. I'll factor out the code to do
the editor thing and make it a separate test.
Thanks,
James
--
James Westby -- GPG Key ID: B577FE13 -- http://jameswestby.net/
seccure key - (3+)k7|M*edCX/.A:n*N!>|&7U.L#9E)Tu)T0>AM - secp256r1/nistp256
More information about the bazaar
mailing list