On Sat, Mar 21, 2020 at 03:50:55PM -0300, Matheus Tavares Bernardino wrote: > > which seems reasonable. But: > > > > ./t1234-foo.sh -rr 5 6 > > > > would get garbled. > > ... we could prohibit using more than one "short option with > arguments" in the same bundle. This would not only solve the problem > for "-rr 5 6"[1] but also for the scenario of future new options. And > it's quite simple to implement, we just have to check whether > $store_arg_to is set before setting it to another value. I'll try that > for v2. Yeah, I'd be perfectly happy with that. This bundling is a new format that did not exist before, so we are not taking away anything you could previously do. As long as we don't produce a wrong or confusing result (and instead say "don't do that; we don't support it", anybody else is free to come along later and make it work. :) > [1]: Someone that used '-rr 5 6' might have wanted the script to run > *both* tests 5 and 6. But I don't think we need to support that now, > since '-r 5 -r 6' doesn't do that as well (instead, the last value > overrides all previous ones). Heh, that's what I assumed "-r 5 -r 6" would do, but I guess it goes to show that I do not use that option very much. :) -Peff
