On Tue, Jul 7, 2015 at 9:24 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Jacob Keller <jacob.e.keller@xxxxxxxxx> writes: > >> +remote.<name>.arbitrarypattern:: >> + When set to true, fetching from this remote will allow arbitrary complex >> + patterns for the fetch refspecs. For example, >> + refs/tags/prefix-*:refs/tags/prefix-* will work as expected. Care should be >> + taken as you could fetch refs into names that don't exist on the remote and >> + may end up pushing them again later by accident. > > Bad name and explanation, unless you truly mean "arbitrary", like > taking something like "refs/ta*/prefix-*-*-foo:refs/*". > I couldn't figure out what to use, and the original intent was to add an option.. but see below, > More importantly, this is not "pattern"; you are talking about > refspec, I think. > In this case the option was an additional modifier to the refspec_patterns, and I was talking about how the pattern could be slightly more arbitrary. I do agree it is a bad name, but i couldn't actually come up with anything better. > But that probably does not matter. I do not think this even needs > to exist as an option. > Yes, I agree especially as I look at it more. I'll work up a patch version which does this without the option. > People's existing settings would not have anything other than an > asterisk as a whole path component on each side (or no asterisk > anywhere), and if they had an asterisk anywhere else they would have > gotten an error and wouldn't have made any progress until they fixed > it. So if you loosen the current rule sligntly and tell them "If > your refspec has an asterisk in it, then you must have one asterisk > on each side of it. That rule hasn't changed. But your asterisks no > longer have to be a whole path component", such a change would not > hurt them. Their existing setting that work would not notice, and > existing users would not be relying on a refspec with an asterisk as > part of a path component to error out. > Right. We aren't breaking anyones current functionality, just adding new functionality. We already check for a * in both sides, and my code will ensure only 1 star total. It will then work for the new somewhat more expanded patterns and we don't need an option. I'll work up a v2. Regards, Jake -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html