> Neil Roberts <bpeeluk@xxxxxxxxxxx> writes: > >> Is this patch going to get applied? Junio C Hamano <gitster@xxxxxxxxx> writes: > I do not think it is likely to happen for a patch without much > comments nor progress after this long blank period, without a > refresher discussion. > > It definitely won't be applied silently in its original form, > especially because the final comment in the old discussion on the > patch in question began with "One could _update_ ..." from the author > of the patch, and then nothing happened. > > http://thread.gmane.org/gmane.comp.version-control.git/122418/focus=122466 Ok, fair enough. I submitted another patch to mailing list earlier which at least addresses the issue mentioned by the original author when he says "One could _update_ ...". > I actually liked the much simpler one by Andreas in the original > thread, but if you really want to use a regexp (which we didn't have > to) we should make it configurable. See the neighbouring discussion > here as well. > > http://thread.gmane.org/gmane.comp.version-control.git/123322 Oh I didn't see that thread, sorry. It's quite tricky to track the issue when it is spread across multiple threads in a mailing list. I'm not particularly set on the idea of it being a regular expression so I'd be happy with an improved version of the existing loop. I'd certainly be happy with it being an option as in your patch here: http://article.gmane.org/gmane.comp.version-control.git/123340 If it is an option as in that patch surely it's quite safe as it can't affect anyone's existing workflow? It might also be nice if it was possible to change it in .git/config so you could enable it by default for projects that use the '[topic]' syntax (such as Cairo and Clutter). > I think we all agree that the behaviour should be improved, but I > think neither Roger's patch nor Andreas's one was the solution.. > People who care need to carry discussions and proposed patches forward > to help us agree on an acceptable solution. Ok, well I do care about this issue and it annoys me regularly so I would love to reopen the discussion. What are the issues with the last patch mentioned above? - Neil -- 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