Re: [PATCH v2 1/3] test-lib: allow selecting tests by substring/regex with --run
- From
Jeff King <peff@peff.net>
- Date
- Oct 14, 2020, 19:49 UTC
- Message-ID
- <20201014194926.GB365553@coredump.intra.peff.net>
- In-Reply-To
- <CABPp-BE0_u=TAJKUGUTLrZbg8vk2MFYmfscT_+MgYwAS2rV8Aw@mail.gmail.com>
On Wed, Oct 14, 2020 at 12:41:00PM -0700, Elijah Newren wrote:
Show 14 quoted lines
> > I use "?" for the same purpose for globs. For things that are casual, > > I find that > > it tends to make the end-user (read: my) experience simpler to use globs than > > to use regexp, largely for your ".*" vs "*" reasons. > > Oh, I thought you were arguing for globs over regexes here just due to > performance reasons. Most of my uses of this feature so far are just > substring matches, so either globs or regexes would be fine for what > I've done so far. Mostly, I was leaning towards regexes because I > figured they were a bit more flexible and perhaps there might arise a > usecase where that would be handy (plus Peff seemed to like them). > But if you find the normal usecase easier with globs, maybe I should > let you and Peff argue it out and then I'll resubmit with the > preferred solution?
Thinking on it more, globs are probably a more natural fit here. Especially if we do eventually allow selection of scripts by name, too (e.g., for GIT_SKIP_TESTS), matching those filenames with a glob would match what happens on the command-line.
-Peff