Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-31 Thread Johannes Schindelin
Hi Junio, On Mon, 29 Aug 2016, Junio C Hamano wrote: > Johannes Schindelin writes: > > > When third-party tools need to access to contents of blobs in the > > database, they might be more interested in the worktree version than in > > the "clean" version of said

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-29 Thread Junio C Hamano
Johannes Schindelin writes: > When third-party tools need to access to contents of blobs in the > database, they might be more interested in the worktree version than in > the "clean" version of said contents. Just a friendly reminder before you completely shift your

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Junio C Hamano
Jeff King writes: > I don't suppose anybody cares that much either way, but it feels weird > to behave differently depending on how we looked up the blob (whereas > for the HEAD:t case, a tree is always a tree). I do not care strongly either way, but HEAD:RelNotes case we _know_

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Jeff King
On Wed, Aug 24, 2016 at 10:02:39AM -0700, Junio C Hamano wrote: > Jeff King writes: > > > On Wed, Aug 24, 2016 at 09:09:06AM -0700, Junio C Hamano wrote: > > > >> > + if (!path) > >> > + path = obj_context.path; > >> > + else if (obj_context.mode ==

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Junio C Hamano
Jeff King writes: > On Wed, Aug 24, 2016 at 09:09:06AM -0700, Junio C Hamano wrote: > >> > + if (!path) >> > + path = obj_context.path; >> > + else if (obj_context.mode == S_IFINVALID) >> > + obj_context.mode = 0100644; >> > + >> >buf = NULL; >> >

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Jeff King
On Wed, Aug 24, 2016 at 09:09:06AM -0700, Junio C Hamano wrote: > > + if (!path) > > + path = obj_context.path; > > + else if (obj_context.mode == S_IFINVALID) > > + obj_context.mode = 0100644; > > + > > buf = NULL; > > switch (opt) { > > case 't': > > The

Re: [PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Junio C Hamano
Johannes Schindelin writes: > diff --git a/builtin/cat-file.c b/builtin/cat-file.c > index 5f91cf4..f8a3a08 100644 > --- a/builtin/cat-file.c > +++ b/builtin/cat-file.c > @@ -61,6 +61,7 @@ static int cat_one_file(int opt, const char *exp_type, > const char

[PATCH v2 0/4] cat-file: optionally convert to worktree version

2016-08-24 Thread Johannes Schindelin
When third-party tools need to access to contents of blobs in the database, they might be more interested in the worktree version than in the "clean" version of said contents. This branch introduces the --filters option to make that happen, the --use-path option to provide the path separately if