Re: [Suggestion] Add Skip for t9020

From: Todd Zullinger <tmz@pobox.com>
Date: Feb 21, 2019, 20:58 UTC
Message-ID: <20190221205837.GX16125@zaya.teonanacatl.net>
In-Reply-To: <004b01d4ca23$faebcad0$f0c36070$@nexbridge.com>

Hi,

Randall S. Becker wrote:

Show 9 quoted lines

> On February 21, 2019 15:00, I wrote:
>> t9020 subtests 1,2,5,6 failed - Not new. unsurprising as there is no SVN or
>> perl with SVN module on platform. It might be useful to have a detection to
>> skip of Perl SVN is not present.
> 
> While this is a bit of a hack, it might be useful for skipping t9020 in
> environments where the svn.remote package is not installed. I can make this
> into a patch if this style is reasonable - guessing probably not and that
> the REMOTE_SVN test should go elsewhere if it is called that.

Jeff King sent an RFC patch which would remove this test and the rest of the vcs-svn experiment in August[1]. Jonathan Nieder replied as one user who would rather see it moved to contrib/, so it was held off.

Whether that has any impact on adding a way to skip all the tests here, I don't know. Maybe it's a gentle nudge in favor of moving them to contrib?

[1] https://public-inbox.org/git/20180817190310.GA5360@sigill.intra.peff.net/

-- 
Todd