On Sun, Jan 27, 2019 at 3:51 AM Sebastian Staudt <korak...@gmail.com> wrote: > > This ensures the given working tree is used for --dirty. > > The implementation of --broken uses diff-index which calls > setup_work_tree() itself.
It would be nice to have a test case covering --broken even if no fix is needed (so that somebody else will not accidentally break it later). I did a quick test and thought it was broken, but it turns out I tested it wrong :P > Signed-off-by: Sebastian Staudt <korak...@gmail.com> > --- > builtin/describe.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/builtin/describe.c b/builtin/describe.c > index cc118448ee..b5b7abdc8f 100644 > --- a/builtin/describe.c > +++ b/builtin/describe.c > @@ -629,6 +629,7 @@ int cmd_describe(int argc, const char **argv, const char > *prefix) > struct argv_array args = ARGV_ARRAY_INIT; > int fd, result; > > + setup_work_tree(); > read_cache(); > refresh_index(&the_index, > REFRESH_QUIET|REFRESH_UNMERGED, > NULL, NULL, NULL); > -- > 2.20.1 > -- Duy
