[PATCH 00/24] Kill the_index part3

2018-08-13 Thread Nguyễn Thái Ngọc Duy
This is the third part of killing the_index (at least outside builtin/). Part 1 [1] is dropped. Part 2 is nd/no-extern on 'pu'. This part is built on top of nd/no-extern. This series would actually break 'pu' because builtin/stash.c uses three functions that are updated here. So we would need some

Re: [PATCH 00/24] Kill the_index part3

2018-08-13 Thread Brandon Williams
On 08/13, Nguyễn Thái Ngọc Duy wrote: > This is the third part of killing the_index (at least outside > builtin/). Part 1 [1] is dropped. Part 2 is nd/no-extern on 'pu'. This > part is built on top of nd/no-extern. > > This series would actually break 'pu' because builtin/stash.c uses > three func

Re: [PATCH 00/24] Kill the_index part3

2018-08-13 Thread Junio C Hamano
Brandon Williams writes: > On 08/13, Nguyễn Thái Ngọc Duy wrote: >> This is the third part of killing the_index (at least outside >> builtin/). Part 1 [1] is dropped. Part 2 is nd/no-extern on 'pu'. This >> part is built on top of nd/no-extern. >> >> This series would actually break 'pu' because

Re: [PATCH 00/24] Kill the_index part3

2018-08-15 Thread Stefan Beller
On Mon, Aug 13, 2018 at 2:24 PM Junio C Hamano wrote: > > Brandon Williams writes: > > > On 08/13, Nguyễn Thái Ngọc Duy wrote: > >> This is the third part of killing the_index (at least outside > >> builtin/). Part 1 [1] is dropped. Part 2 is nd/no-extern on 'pu'. This > >> part is built on top o