Re: [PATCH v3 1/3] test-lib: allow selecting tests by substring/glob with --run
On Fri, Oct 16, 2020 at 01:02:16PM -0700, Junio C Hamano wrote:
Show 17 quoted lines
> Elijah Newren <newren@gmail.com> writes:
>
> > Given that t/README explicitly shows examples of space-separated lists
> > of numbers, I'm worried we're breaking long-built expectations of
> > other developers by changing IFS here. Perhaps I could instead add
> > the following paragraph to t/README:
>
> Unlike something that would affect end-users, I think it is OK to
> break backward compatibility one-way. If you suddenly forbid spaces
> and force our developers to use comma and nothing else, in muscle
> memory of their fingers and/or in their scripts, in a version merged
> to 'master', as long as their new habit and updated scripts use
> comma consistently, they work fine on 'maint', right?
>
> If there is no such "works on both sides of the flag day" choice, it
> is a different story, of course, but comma should work fine for us
> in this case.
I was just about to write the same argument. :)
-Peff