Not modifying a suggested commit message commits anyway
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Fix Released
|
Medium
|
Robert Collins |
Bug Description
The ability to seed the commit message with text taken from elsewhere is useful, but
it interacts badly with the desire to cancel commits at that point.
Often when editing a commit message it makes me realise something is missing, and
so I want to cancel the commit.
Normally I can do this by exiting without saving, or saving and closing my editor without
writing anything.
However, when the commit has been seeded, neither of these works, the only way to do
it is to delete all the text in the file, then save that and exit. Doing anything else results in
the commit happening anyway, so I have to uncommit (and sometimes don't notice until later)
Jelmer's suggestion was to have bzr prompt if the commit message is unchanged from the
seeded one.
Thanks,
James
Changed in bzr: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in bzr: | |
status: | Confirmed → In Progress |
assignee: | nobody → Robert Collins (lifeless) |
Changed in bzr: | |
milestone: | none → 2.2b4 |
status: | In Progress → Fix Released |
Changed in bzr: | |
assignee: | nobody → Robert Collins (lifeless) |
status: | Confirmed → Fix Released |
On Mon, 2010-03-01 at 19:50 +0000, James Westby wrote:
> Normally I can do this by exiting without saving, or saving and closing my editor without
> writing anything.
>
> However, when the commit has been seeded, neither of these works, the only way to do
> it is to delete all the text in the file, then save that and exit. Doing anything else results in
> the commit happening anyway, so I have to uncommit (and sometimes don't notice until later)
>
> Jelmer's suggestion was to have bzr prompt if the commit message is unchanged from the
> seeded one.
That won't work well if bzr can't prompt (it can run EDITOR under e.g. X
even if bzr has no stdin).
Perhaps having a line
**** DELETE THIS LINE to use the suggested commit message ****
?
-Rob