On 09/27, Junio C Hamano wrote:
> Brandon Williams <bmw...@google.com> writes:
> 
> >     if (recurse_submodules &&
> > -       (show_stage || show_deleted || show_others || show_unmerged ||
> > +       (show_deleted || show_others || show_unmerged ||
> >          show_killed || show_modified || show_resolve_undo ||
> > -        show_valid_bit || show_tag || show_eol || with_tree ||
> > -        (line_terminator == '\0')))
> > +        with_tree))
> >             die("ls-files --recurse-submodules unsupported mode");
> 
> Ahh, one more thing, and this comment probably applies to 2/4 not
> this one, but if the intention is to shrink this "not supported yet"
> check as the series progresses, in the earlier step the check would
> need to make sure no pathspec is given, which is first supported in
> 4/4, I think.  It is not a big deal to require rerolling by itself,
> bit if you are rerolling 2/4 for other reasons, then...

Oh there is a separate if gaurd for pathspecs which is introduced in 2/4
and then removed once pathspec support has been added in 4/4.

-- 
Brandon Williams

Reply via email to