Re: [PATCH v2 05/10] replace: make sure --edit results in a different object

2014-05-17 Thread Jeff King
On Sat, May 17, 2014 at 08:41:27AM +0200, Christian Couder wrote: > It's a bad idea to create a replace ref for an object > that points to the original object itself. > > That's why we have to check if the result from editing > the original object is a different object and error out > if it isn't

[PATCH v2 05/10] replace: make sure --edit results in a different object

2014-05-16 Thread Christian Couder
It's a bad idea to create a replace ref for an object that points to the original object itself. That's why we have to check if the result from editing the original object is a different object and error out if it isn't. Signed-off-by: Christian Couder --- builtin/replace.c | 3 +++ 1 file chan