From: Junio C Hamano Date: Mon, 02 Dec 2019 17:01:55 GMT Subject: Re: [PATCH v2] t4210: skip i18n tests that don't work on FreeBSD Message-ID: In-Reply-To: <20191127172035.GB1123@sigill.intra.peff.net> Jeff King writes: > On Wed, Nov 27, 2019 at 05:15:07PM +0000, Ed Maste wrote: > >> Extend test-lib.sh to add a REGEX_ILLSEQ prereq, set it on FreeBSD, and >> add !REGEX_ILLSEQ to the two affected tests. > > Thanks, this is much nicer. > >> +FreeBSD) >> + test_set_prereq REGEX_ILLSEQ >> + test_set_prereq POSIXPERM >> + test_set_prereq BSLASHPSPEC >> + test_set_prereq EXECKEEPSPID >> + ;; > > I scratched my head at these for a minute, but I see you are just > covering the bits set in the "*" case that we now no longer trigger. > > It would probably be cleaner to set them ahead of time and just unset > them selectively in MINGW, etc. But we don't have any way to unset a > prereq, so lets' go with this for now. :) Thanks, both. Queued. Will merge to 'next' shortly.