[PATCH 9 of 9] cmdutil: omit redundant "savecommitmessage()" in"tryimportone()""

FUJIWARA Katsunori foozy at lares.dti.ne.jp
Wed May 7 14:15:29 UTC 2014


At Tue, 06 May 2014 15:12:28 -0500,
Sean Farley wrote:
> 
> 
> FUJIWARA Katsunori <foozy at lares.dti.ne.jp> writes:
> 
> > # HG changeset patch
> > # User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
> > # Date 1399292800 -32400
> > #      Mon May 05 21:26:40 2014 +0900
> > # Branch stable
> > # Node ID ad9e8d71ce784496fe98e5fd33f18af30787db67
> > # Parent  24416f80f94093cfb1f9699dfa09d794b194db53
> > cmdutil: omit redundant "savecommitmessage()" in "tryimportone()"
> 
> I like this series :-) A few comments / questions (that shouldn't affect
> accepting these patches):
>
> - is this work paving the way for reusing a previous commit message,
>   say, in 'hg ci --amend' without having to launch the editor?

Do you mean some kind of automation around reusing
".hg/last-message.txt" and invocation of commit message editor ?

If so, this series doesn't focus on paving the way for it.


> - are you considering moving localrepo.savecommitmessage to
>   committablectx? (see below for my motivation)

Oh, I don't notice that refactoring point !

My direction is just to (1) make all (commit log like) message
editing use one of "commiteditor" or "commitforceeditor" (2) in the
simple way.

  (1) now, some commands still use "ui.edit()" directly

  (2) replace explicit "opts['edit']" checks and direct references to
      "commit*editor" by introducing the specific utility function

So, patch #6 in this series is only one for "context.py" around this
working.


> - (marmoute, mpm) this series conflicts with my work on memctx and
>   moving localrepo.commit to committablectx.commit but that's not a big
>   deal; I can rebase my work on top of this series and send my patches
>   afterwards


----------------------------------------------------------------------
[FUJIWARA Katsunori]                             foozy at lares.dti.ne.jp



More information about the Mercurial-devel mailing list