Subject: Re: [PATCH v3 1/2] t: add a test helper for getting hostname

From: Junio C Hamano <[email-masked]>
Date: Tue, 19 Mar 2024 14:00:34 -0700

Jeff King <[email-masked]> writes:

> I think you need to lose the exit status. Or alternatively do:
>
>   hostname=$(test-tool xgethostname || echo unknown)
>
> See my other reply.

As "test-tool xgethostname" runs exactly the same codepath as
"includeIf hostname:blah" feature, I would actually prefer for a
failing "test-tool gethostname" to _break_ this test so that people
can take notice.