On Tue, 2020-10-06 at 08:01 -0600, Jonathan Corbet wrote: > On Tue, 6 Oct 2020 08:42:07 +0200 > Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx> wrote: > > > As right now we don't support Sphinx version 3.0[1], we're actually using just > > $sphinx_major. So, I'm wonder if it would make sense to also make <minor> > > optional. > > Maybe...someday we may need it, knowing how the Sphinx folks approach > compatibility, but I guess we can always add it then if so. > > > The change would be trivial, although the regex will become even more > > harder to read ;-) > > ^(\d+)(\.(\d+)){,2} > > ? (untested, of course) This doesn't work as capture groups don't nest using {m, n}