changeset 9935:48b81d9bca8d

commit: if relevant, tell user their commit message was saved. (issue1635) Specifically, if: 1) the user edited the message (it didn't come straight from -m) and 2) the commit was aborted by an exception then the saved commit message in .hg/last-message.txt could come in handy, so mention it with a ui.write(). This doesn't help users who manually rollback to amend a changeset: the fact that the message was saved to .hg/last-message.txt is invisible in that case.
author Greg Ward <greg-hg@gerg.ca>
date Tue, 24 Nov 2009 21:08:40 -0500
parents 720f70b720d3
children 3aa420c56ac4
files mercurial/localrepo.py tests/test-rollback tests/test-rollback.out
diffstat 3 files changed, 26 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/localrepo.py	Tue Nov 24 21:08:39 2009 -0500
+++ b/mercurial/localrepo.py	Tue Nov 24 21:08:40 2009 -0500
@@ -825,6 +825,7 @@
                                       extra, changes)
             if editor:
                 cctx._text = editor(self, cctx, subs)
+            edited = (text != cctx._text)
 
             # commit subs
             if subs:
@@ -844,7 +845,14 @@
             msgfile.write(cctx._text.rstrip() + '\n')
             msgfile.close()
 
-            ret = self.commitctx(cctx, True)
+            try:
+                ret = self.commitctx(cctx, True)
+            except:
+                if edited:
+                    msgfn = self.pathto(msgfile.name[len(self.root)+1:])
+                    self.ui.write(
+                        _('note: commit message saved in %s\n') % msgfn)
+                raise
 
             # update dirstate and mergestate
             for f in changes[0] + changes[1]:
--- a/tests/test-rollback	Tue Nov 24 21:08:39 2009 -0500
+++ b/tests/test-rollback	Tue Nov 24 21:08:40 2009 -0500
@@ -34,6 +34,15 @@
 echo '% rollback by pretxncommit saves commit message (issue 1635)'
 echo a >> a
 hg --config hooks.pretxncommit=/bin/false commit -m"precious commit message"
-
 echo '.hg/last-message.txt:'
 cat .hg/last-message.txt
+
+echo '% same thing, but run $EDITOR'
+cat > $HGTMP/editor <<'__EOF__'
+#!/bin/sh
+echo "another precious commit message" > "$1"
+__EOF__
+chmod +x $HGTMP/editor
+HGEDITOR=$HGTMP/editor hg --config hooks.pretxncommit=/bin/false commit
+echo '.hg/last-message.txt:'
+cat .hg/last-message.txt
--- a/tests/test-rollback.out	Tue Nov 24 21:08:39 2009 -0500
+++ b/tests/test-rollback.out	Tue Nov 24 21:08:40 2009 -0500
@@ -34,3 +34,10 @@
 abort: pretxncommit hook exited with status 1
 .hg/last-message.txt:
 precious commit message
+% same thing, but run $EDITOR
+transaction abort!
+rollback completed
+note: commit message saved in .hg/last-message.txt
+abort: pretxncommit hook exited with status 1
+.hg/last-message.txt:
+another precious commit message