Not modifying a suggested commit message commits anyway

Bug #530265 reported by James Westby
6
This bug affects 1 person
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

Tags: udd
Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 530265] [NEW] Not modifying a suggested commit message commits anyway

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

Martin Pool (mbp)
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
Revision history for this message
Robert Collins (lifeless) wrote :

Sorry, that branch appears to have done something totally different. ENOIDEAWHY.

Changed in bzr:
status: Fix Released → Confirmed
assignee: Robert Collins (lifeless) → nobody
John A Meinel (jameinel)
Changed in bzr:
assignee: nobody → Robert Collins (lifeless)
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.