Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-23 Thread Thomas Rast
Junio C Hamano writes: > * tr/test-v-and-v-subtest-only (2013-06-18) 8 commits > - test-lib: support running tests under valgrind in parallel > - test-lib: allow prefixing a custom string before "ok N" etc. > - test-lib: valgrind for only tests matching a pattern > - test-lib: verbose mode fo

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > Because the implementation is too ugly. I know :) The only reason I sent it with code is because I didn't get any responses to an email without code. If you agree that it is a useful feature, we have to figure out a beautiful implementation. > I would however can imagine

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Junio C Hamano
Ramkumar Ramachandra writes: > Junio C Hamano wrote: >>> - rr/describe-contains-all at $gmane/228278. >> >> This may overlap with a topic in flight (but I didn't look at it). > > Let me know if I can do anything to make it easier for you. I'm quite > excited about this one. > >>> - rr/mixed-cas

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > Unfortunately, I tend to become bottleneck more often than you do, > so I do not think that would be a good use of my time. Besides, as Antoine points out, those numbers might well be useless (or worse, misleading). It's probably not worth the effort. > Which ones went un

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Junio C Hamano
Ramkumar Ramachandra writes: > Junio C Hamano wrote: >>> ... I've noticed a significant decline in >>> list-reviews in general: a lot of good code on the list hasn't been >>> reviewed. >> >> Hmph, I do not particularly see that happening. > > Observer bias. To verify either claim, could you run

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Antoine Pelisse
On Fri, Jun 21, 2013 at 7:13 PM, Ramkumar Ramachandra wrote: > Junio C Hamano wrote: >>> ... I've noticed a significant decline in >>> list-reviews in general: a lot of good code on the list hasn't been >>> reviewed. >> >> Hmph, I do not particularly see that happening. > > Observer bias. To ver

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Ramkumar Ramachandra
Junio C Hamano wrote: >> ... I've noticed a significant decline in >> list-reviews in general: a lot of good code on the list hasn't been >> reviewed. > > Hmph, I do not particularly see that happening. Observer bias. To verify either claim, could you run some stats on the ML archives [1] and re

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Junio C Hamano
Ramkumar Ramachandra writes: > ... I've noticed a significant decline in > list-reviews in general: a lot of good code on the list hasn't been > reviewed. Hmph, I do not particularly see that happening. I actually think all the usual suspects (I could name them, but I wouldn't---you know who y

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Ramkumar Ramachandra
Junio C Hamano wrote: >> - rr/describe-contains-all at $gmane/228278. > > This may overlap with a topic in flight (but I didn't look at it). Let me know if I can do anything to make it easier for you. I'm quite excited about this one. >> - rr/mixed-case-aliases at $gmane/227959. > > Personally,

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Junio C Hamano
Ramkumar Ramachandra writes: > Four topics are awaiting review (they're important to me in this order): > > - rr/for-each-ref-pretty at $gmane/227057. No comment on this at this moment. > - rr/describe-contains-all at $gmane/228278. This may overlap with a topic in flight (but I didn't look a

Re: What's cooking in git.git (Jun 2013, #07; Thu, 20)

2013-06-21 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > * rr/triangle-push-fix (2013-06-20) 9 commits > - push: honor branch.*.push > - SQUASH??? fix git-config push.default description > - SQUASH??? minimum "simple" safety fix-up > - t/t5528-push-default: test pushdefault workflows > - t/t5528-push-default: generalize test_