Re: [PATCH] t/README: mention test files are make targets

From: Chris Torek <chris.torek@gmail.com>
Date: Mar 25, 2024, 09:59 UTC
Message-ID: <CAPx1GvcPEeC3HTdb4bUYASz-t=a7Z2MEUamB4QyO44y4MRy6Pw@mail.gmail.com>
In-Reply-To: <xmqqfrwfp0hh.fsf@gitster.g>

On Mon, Mar 25, 2024 at 2:49 AM Junio C Hamano <gitster@pobox.com> wrote:

> As long as we do not use these $(T) files as an input to some other
> thing and list them as the dependencies, we are OK, though.

You could (maybe later / at need) stop listing them as `.PHONY` and instead use:

    $(T)::
        sh -c ./$@

or similar, so that some $(T) *can* be an input. Note that this requires using double-colon rules earlier to build the test.

I wouldn't do this without a pretty strong reason though.

Chris