Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-24 Thread Johannes Schindelin
Hi Peff, On Fri, 21 Jun 2019, Jeff King wrote: > On Fri, Jun 21, 2019 at 03:16:52PM +0200, Johannes Schindelin wrote: > > > > I think your warning in the manpage that this is for debugging is fine, > > > as it does not put us on the hook for maintaining the feature nor its > > > format forever. W

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-24 Thread Duy Nguyen
On Mon, Jun 24, 2019 at 4:32 PM Johannes Schindelin wrote: > > Hi Duy, > > On Fri, 21 Jun 2019, Duy Nguyen wrote: > > > On Fri, Jun 21, 2019 at 8:16 PM Johannes Schindelin > > wrote: > > > > > > I think your warning in the manpage that this is for debugging is fine, > > > > as it does not put us

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-24 Thread Johannes Schindelin
Hi Peff & Junio, On Fri, 21 Jun 2019, Jeff King wrote: > On Fri, Jun 21, 2019 at 08:10:58AM -0700, Junio C Hamano wrote: > > > Duy Nguyen writes: > > > > > Considering the amount of code to output these, supporting multiple > > > formats would be a nightmare. I may be ok with versioning the outp

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-24 Thread Johannes Schindelin
Hi Duy, On Fri, 21 Jun 2019, Duy Nguyen wrote: > On Fri, Jun 21, 2019 at 8:16 PM Johannes Schindelin > wrote: > > > > I think your warning in the manpage that this is for debugging is fine, > > > as it does not put us on the hook for maintaining the feature nor its > > > format forever. We might

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Duy Nguyen
On Sat, Jun 22, 2019 at 6:31 AM brian m. carlson wrote: > > On 2019-06-19 at 09:58:50, Nguyễn Thái Ngọc Duy wrote: > > This is probably just my itch. Every time I have to do something with > > the index, I need to add a little bit code here, a little bit there to > > get a better "view" of the ind

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread brian m. carlson
On 2019-06-19 at 09:58:50, Nguyễn Thái Ngọc Duy wrote: > This is probably just my itch. Every time I have to do something with > the index, I need to add a little bit code here, a little bit there to > get a better "view" of the index. > > This solves it for me. It allows me to see pretty much eve

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Jeff King
On Fri, Jun 21, 2019 at 08:10:58AM -0700, Junio C Hamano wrote: > Duy Nguyen writes: > > > Considering the amount of code to output these, supporting multiple > > formats would be a nightmare. I may be ok with versioning the output > > so the tool know what format they need to deal with, but I'd

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Jeff King
On Fri, Jun 21, 2019 at 03:16:52PM +0200, Johannes Schindelin wrote: > > I think your warning in the manpage that this is for debugging is fine, > > as it does not put us on the hook for maintaining the feature nor its > > format forever. We might want to call it "--debug=json" or something, > > t

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Jeff King
On Fri, Jun 21, 2019 at 03:37:45PM +0700, Duy Nguyen wrote: > On Thu, Jun 20, 2019 at 2:17 AM Jeff King wrote: > > I think your warning in the manpage that this is for debugging is fine, > > as it does not put us on the hook for maintaining the feature nor its > > format forever. We might want to

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Junio C Hamano
Duy Nguyen writes: > Considering the amount of code to output these, supporting multiple > formats would be a nightmare. I may be ok with versioning the output > so the tool know what format they need to deal with, but I'd rather > support just one version. For third parties wanting to dig deep,

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Duy Nguyen
On Fri, Jun 21, 2019 at 8:16 PM Johannes Schindelin wrote: > > I think your warning in the manpage that this is for debugging is fine, > > as it does not put us on the hook for maintaining the feature nor its > > format forever. We might want to call it "--debug=json" or something, > > though, in

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Johannes Schindelin
Hi Peff, On Wed, 19 Jun 2019, Jeff King wrote: > On Wed, Jun 19, 2019 at 04:58:50PM +0700, Nguyễn Thái Ngọc Duy wrote: > > > This is probably just my itch. Every time I have to do something with > > the index, I need to add a little bit code here, a little bit there to > > get a better "view" of

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-21 Thread Duy Nguyen
On Thu, Jun 20, 2019 at 2:17 AM Jeff King wrote: > I think your warning in the manpage that this is for debugging is fine, > as it does not put us on the hook for maintaining the feature nor its > format forever. We might want to call it "--debug=json" or something, Hmm.. does it mean we make --d

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-20 Thread Jeff Hostetler
On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote: This is probably just my itch. Every time I have to do something with the index, I need to add a little bit code here, a little bit there to get a better "view" of the index. This solves it for me. It allows me to see pretty much everything in

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > This is probably just my itch. Every time I have to do something with > the index, I need to add a little bit code here, a little bit there to > get a better "view" of the index. ;-) JSON is not particularly my cup-of-tea but it is better than many other things

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Jeff King
On Wed, Jun 19, 2019 at 04:58:50PM +0700, Nguyễn Thái Ngọc Duy wrote: > This is probably just my itch. Every time I have to do something with > the index, I need to add a little bit code here, a little bit there to > get a better "view" of the index. > > This solves it for me. It allows me to see

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Derrick Stolee
On 6/19/2019 8:42 AM, Duy Nguyen wrote: > On Wed, Jun 19, 2019 at 6:58 PM Derrick Stolee wrote: >> >> On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote: >>> This is probably just my itch. Every time I have to do something with >>> the index, I need to add a little bit code here, a little bit there

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Duy Nguyen
On Wed, Jun 19, 2019 at 6:58 PM Derrick Stolee wrote: > > On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote: > > This is probably just my itch. Every time I have to do something with > > the index, I need to add a little bit code here, a little bit there to > > get a better "view" of the index. > >

Re: [PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Derrick Stolee
On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote: > This is probably just my itch. Every time I have to do something with > the index, I need to add a little bit code here, a little bit there to > get a better "view" of the index. > > This solves it for me. It allows me to see pretty much everythi

[PATCH 0/8] Add 'ls-files --json' to dump the index in json

2019-06-19 Thread Nguyễn Thái Ngọc Duy
This is probably just my itch. Every time I have to do something with the index, I need to add a little bit code here, a little bit there to get a better "view" of the index. This solves it for me. It allows me to see pretty much everything in the index (except really low detail stuff like pathnam