[PATCH v2 00/10] replace: add option to edit a Git object

2014-05-17 Thread Christian Couder
This patch series comes from what Peff sent in the following thread:

http://thread.gmane.org/gmane.comp.version-control.git/243361/focus=243528

The first 4 patches (1/4, 2/4, 3/4 and 4/4) didn't change since v1.
I added 6 more small patches to add tests, documentation and a few
small improvements. 

Christian Couder (6):
  replace: make sure --edit results in a different object
  replace: refactor checking ref validity
  replace: die early if replace ref already exists
  replace: add tests for --edit
  replace: add --edit to usage string
  Documentation: replace: describe new --edit option

Jeff King (4):
  replace: refactor command-mode determination
  replace: use OPT_CMDMODE to handle modes
  replace: factor object resolution out of replace_object
  replace: add --edit option

 Documentation/git-replace.txt |  15 ++-
 builtin/replace.c | 225 +-
 t/t6050-replace.sh|  29 ++
 3 files changed, 223 insertions(+), 46 deletions(-)

-- 
1.9.rc0.17.g651113e

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/10] replace: add option to edit a Git object

2014-05-17 Thread Jeff King
On Sat, May 17, 2014 at 08:41:22AM +0200, Christian Couder wrote:

 This patch series comes from what Peff sent in the following thread:
 
 http://thread.gmane.org/gmane.comp.version-control.git/243361/focus=243528
 
 The first 4 patches (1/4, 2/4, 3/4 and 4/4) didn't change since v1.
 I added 6 more small patches to add tests, documentation and a few
 small improvements. 

Thanks, I think these look pretty good. I made a few small comments in
reply to the individual patches.

-Peff
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html