Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-11 Thread Junio C Hamano
Duy Nguyen writes: > In the end, there's no variant, only one function that always takes > 'struct repository *' and I wanted to keep the shorter name 'rerere'. > But let's go with adding repo_rerere() and deprecating rerere(). If it > turns out later that repo_rerere is too long (or it's repo_xy

Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-09 Thread Duy Nguyen
On Wed, Sep 5, 2018 at 11:04 PM Junio C Hamano wrote: > > now and here, but at some later date, we would want to > > 'git revert 24/24' after fixing all in-flights of today. > > No. We do not want to revert the whole thing. > > If the function that takes a_repository is called repo_rerere(), as >

Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-05 Thread Junio C Hamano
Stefan Beller writes: > Presumably you merge the tip of this series (which contains 24/24) > with the other in-flight topics, that make new uses of > init_revisions(revs, prefix), which 24/24 allows. Going on either > parent side of such a merge will have working commits, that compile. > > So fro

Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-04 Thread Stefan Beller
On Tue, Sep 4, 2018 at 1:57 PM Junio C Hamano wrote: > > Nguyễn Thái Ngọc Duy writes: > > > The three functions init_revisions(), diff_setup() and rerere() are > > prefixed temporarily with repo_ to avoid breaking other topics which > > add new call sites for these functions. This is a temporary

Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-04 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > The three functions init_revisions(), diff_setup() and rerere() are > prefixed temporarily with repo_ to avoid breaking other topics which > add new call sites for these functions. This is a temporary > measure. Once everything is merged, it will be reverted and th

Re: [PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-04 Thread Stefan Beller
On Mon, Sep 3, 2018 at 11:10 AM Nguyễn Thái Ngọc Duy wrote: > > The three functions init_revisions(), diff_setup() and rerere() are > prefixed temporarily with repo_ to avoid breaking other topics which > add new call sites for these functions. This is a temporary > measure. Once everything is mer

[PATCH v2 24/24] Rename functions to avoid breaking in-flight topics

2018-09-03 Thread Nguyễn Thái Ngọc Duy
The three functions init_revisions(), diff_setup() and rerere() are prefixed temporarily with repo_ to avoid breaking other topics which add new call sites for these functions. This is a temporary measure. Once everything is merged, it will be reverted and the new call sites fixed. Signed-off-by: