Subject: Re: [PATCH] test-lib: try harder to ensure a working jgit

From: Jonathan Nieder <[email-masked]>
Date: Mon, 13 May 2019 19:32:45 -0700

Todd Zullinger wrote:

> The JGIT prereq uses 'type jgit' to determine whether jgit is present.
> While this should be sufficient, if the jgit found is broken we'll waste
> time running tests which fail due to no fault of our own.
>
> Use 'jgit --version' instead, to catch some badly broken jgit
> installations.
>
> Signed-off-by: Todd Zullinger <[email-masked]>
> ---
> I ran into such a broken jgit on Fedora >= 30¹.  This is clearly a
> problem in the Fedora jgit package which will hopefully be resolved
> soon.  But it may be good to avoid wasting time debugging tests which
> fail due to a broken tool outside of our control.
>
> ¹ https://bugzilla.redhat.com/1709624

Reviewed-by: Jonathan Nieder <[email-masked]>

It would be nice to describe that bug in the commit message, to save
readers some head scratching.

Thanks,
Jonathan