Re: [PATCH 0/5] "status --ignored"

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 10, 2010, 18:27 UTC
Message-ID: <7v633zxiw4.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20100410084007.GA16843@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 5 quoted lines

> ... But finding every file means
> we have to traverse areas that git otherwise wouldn't look at, which
> might mean going to disk to pull in all of the "foo/" directory
> structure (which is less likely to be cached, since the rest of git
> isn't touching it), even though it may not even be of interest to us.

Yes, that is why it is adequate for us use COLLECT_IGNORED in "git add" and give "foo/ is outside---you as an intelligent human should be able to infer that your foo/bar is also" without double traversal.

Eric's tool might want the same abbreviated information if it is just relaying our output to an intelligent human. Or it might want to get all paths if it wants to operate on them itself. Knowing VC I chose to illustrate how to do the latter, but in the real implementation, we may want --show-ignored=normal vs --show-ignored=expanded to support both uses.