Re: [PATCH v2 1/2] modernize t7300

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 7, 2015, 22:50 UTC
Message-ID: <xmqqr3ixlvpt.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <CAPig+cQOW69-YNaVvqaVYyNt6cT-u92a=i0V6YufnVEmYHy3LA@mail.gmail.com>

Eric Sunshine <sunshine@sunshineco.com> writes:

> Alternately, update test_path_foo() functions to accept multiple
> pathnames, or is that too ugly?

That actually would have been my first choice, except (1) that path_is_missing has a cruft whose usefulness is dubious, and (2) that "path_exists A B C" and "path_is_missing D E F" would be gramatically incorrect.

I think we should first see if we can remove that "customized message" that appears only in path_is_missing and remove it if we can. Extending "path_is_missing A B C" and friends to work would then become trivial.

Thanks.