Re: [Python-Dev] Appeal for reviews

2014-04-15 Thread Nick Coghlan
On 14 Apr 2014 08:42, R. David Murray rdmur...@bitdance.com wrote: On Mon, 14 Apr 2014 08:18:13 -0400, Nick Coghlan ncogh...@gmail.com wrote: On 14 Apr 2014 01:56, Stephen J. Turnbull step...@xemacs.org wrote: mar...@v.loewis.de writes: For gaining commit access, it's really more

Re: [Python-Dev] Appeal for reviews

2014-04-15 Thread Guido van Rossum
On Tue, Apr 15, 2014 at 10:41 AM, Nick Coghlan ncogh...@gmail.com wrote: On 14 Apr 2014 08:42, R. David Murray rdmur...@bitdance.com wrote: When considering who we give commit access to, I think we would be well served to start giving more weight to the quality of the code reviews that

Re: [Python-Dev] Appeal for reviews

2014-04-15 Thread Stephen J. Turnbull
On 14 Apr 2014 08:42, R. David Murray rdmur...@bitdance.com wrote: Or to put it another way, I'd like to encourage contributors who want to get commit access to focus just as much on doing good reviews as they do on writing new patches. Currently the focus is all on getting patches

Re: [Python-Dev] Appeal for reviews

2014-04-15 Thread Terry Reedy
On 4/15/2014 12:15 PM, Stephen J. Turnbull wrote: I've always really liked MvL's 5-reviews-to-get-1 approach. The only thing I don't like about it[3] is that it puts an explicit price on core developer time (my time is worth 5x as much as yours). Not really true since any of the 5 could be

Re: [Python-Dev] Appeal for reviews

2014-04-15 Thread Stephen J. Turnbull
Terry Reedy writes: On 4/15/2014 12:15 PM, Stephen J. Turnbull wrote: The only thing I don't like about it[3] is that it puts an explicit price on core developer time (my time is worth 5x as much as yours). Not really true But that is *not* your call! It's for the would-be

Re: [Python-Dev] Appeal for reviews

2014-04-14 Thread Nick Coghlan
On 14 Apr 2014 01:56, Stephen J. Turnbull step...@xemacs.org wrote: mar...@v.loewis.de writes: For gaining commit access, it's really more important that the patch is factually finished, than that it's author believes it to. If people get it right the first time often enough, they get

Re: [Python-Dev] Appeal for reviews

2014-04-14 Thread R. David Murray
On Mon, 14 Apr 2014 08:18:13 -0400, Nick Coghlan ncogh...@gmail.com wrote: On 14 Apr 2014 01:56, Stephen J. Turnbull step...@xemacs.org wrote: mar...@v.loewis.de writes: For gaining commit access, it's really more important that the patch is factually finished, than that it's author

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Stephen J. Turnbull
Benjamin Peterson writes: I don't think Nikolaus is wrong to post here. I often tell people that sometimes the only way to get your patches in is to constantly poke us about it. I admit the tone was biased toward nagging or blaming the victim, and again I apologize for causing

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Janzert
On 4/13/2014 2:36 AM, Stephen J. Turnbull wrote: [snip] My claim is that in current circumstances, core-mentorship would be a more *effective* channel because - core-mentorship is *explicitly* for poking Those Who Can Help (among other requests for help); - a surprisingly large (to me,

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Stephen J. Turnbull
Janzert writes: On 4/13/2014 2:36 AM, Stephen J. Turnbull wrote: [snip] My claim is that in current circumstances, core-mentorship would be a more *effective* channel because - core-mentorship is *explicitly* for poking Those Who Can Help (among other requests for help);

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Antoine Pitrou
Le 13/04/2014 03:07, Benjamin Peterson a écrit : On Sat, Apr 12, 2014, at 17:30, Stephen J. Turnbull wrote: I apologize for the tone. I need to go *right* now, and can't fix that. Really, I'm sympathetic and my goal is not just to defend python-dev, but to help you get the reviews your work

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Martin v. Löwis
Am 13.04.14 03:07, schrieb Benjamin Peterson: On Sat, Apr 12, 2014, at 17:30, Stephen J. Turnbull wrote: I apologize for the tone. I need to go *right* now, and can't fix that. Really, I'm sympathetic and my goal is not just to defend python-dev, but to help you get the reviews your work

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Martin v. Löwis
Am 13.04.14 08:36, schrieb Stephen J. Turnbull: I admit the tone was biased toward nagging or blaming the victim, and again I apologize for causing misunderstanding. Nikolaus isn't wrong for posting here. My claim is that in current circumstances, core-mentorship would be a more *effective*

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Terry Reedy
On 4/13/2014 2:46 PM, Martin v. Löwis wrote: As for the request Are you sure that the patch is ready: this is *very* difficult to answer for the author. We all have experienced that patches that we considered good were critized out of nowhere, and I just did the same to Nikolaus. There is just

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Nikolaus Rath
Terry Reedy tjre...@udel.edu writes: [Quote conveniently rearranged] I've accumulated a number of patches in the issue tracker that are waiting for someone to review/commit/reject them. I'm eager to make corrections as necessary, I just need someone to look the work that I've done so far: Do

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Nikolaus Rath
Stephen J. Turnbull step...@xemacs.org writes: I apologize for the tone. I need to go *right* now, and can't fix that. Really, I'm sympathetic and my goal is not just to defend python-dev, but to help you get the reviews your work deserves. Please read with that in mind. Will do - but why

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Nikolaus Rath
Martin v. Löwis mar...@v.loewis.de writes: Am 13.04.14 08:36, schrieb Stephen J. Turnbull: I admit the tone was biased toward nagging or blaming the victim, and again I apologize for causing misunderstanding. Nikolaus isn't wrong for posting here. My claim is that in current circumstances,

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Stephen J. Turnbull
Nikolaus Rath writes: I've described the status of each bug in more detail in my reply Terry, but generally all the issues contain a testcase (i.e., so I consider them confirmed), do not require a PEP, contain a patch that needs review and include documentation updates. I have signed the

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Stephen J. Turnbull
Martin v. Löwis writes: Am 13.04.14 08:36, schrieb Stephen J. Turnbull: - core-mentorship is *explicitly* for poking Those Who Can Help (among other requests for help); It would be worth an experiment. I know that I wouldn't have reviewed Nikolaus' patches if he had posted to

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread martin
Quoting Stephen J. Turnbull step...@xemacs.org: Your modesty is not in question. :-) An explicit statement that These are ready allows an experienced developer to give you feedback not only about whether the patches are in fact ready, but whether your judgment about patches is ready for

Re: [Python-Dev] Appeal for reviews

2014-04-13 Thread Stephen J. Turnbull
mar...@v.loewis.de writes: For gaining commit access, it's really more important that the patch is factually finished, than that it's author believes it to. If people get it right the first time often enough, they get commit access. Yes, that's what I had in mind, but I guess I explained

[Python-Dev] Appeal for reviews

2014-04-12 Thread Nikolaus Rath
Hello, I've accumulated a number of patches in the issue tracker that are waiting for someone to review/commit/reject them. I'm eager to make corrections as necessary, I just need someone to look the work that I've done so far: * http://bugs.python.org/issue20951 (SSLSocket.send() returns 0 for

Re: [Python-Dev] Appeal for reviews

2014-04-12 Thread Terry Reedy
On 4/12/2014 2:58 PM, Nikolaus Rath wrote: I've accumulated a number of patches in the issue tracker that are waiting for someone to review/commit/reject them. I'm eager to make corrections as necessary, I just need someone to look the work that I've done so far: If I did not have several

Re: [Python-Dev] Appeal for reviews

2014-04-12 Thread Nick Coghlan
On 12 Apr 2014 18:08, Terry Reedy tjre...@udel.edu wrote: On 4/12/2014 2:58 PM, Nikolaus Rath wrote: I realize that core developer time is scarce, so I have started to only work on patches after I've confirmed that someone is available and interested to review them. However, it would be great

[Python-Dev] Appeal for reviews

2014-04-12 Thread Stephen J. Turnbull
I apologize for the tone. I need to go *right* now, and can't fix that. Really, I'm sympathetic and my goal is not just to defend python-dev, but to help you get the reviews your work deserves. Please read with that in mind. Steve Nikolaus Rath writes: I've accumulated a number of patches

Re: [Python-Dev] Appeal for reviews

2014-04-12 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/12/2014 08:30 PM, Stephen J. Turnbull wrote: it's a matter of time before the contribution is integrated. Our current backlog is bad enough that many contributions are effectively wasted: they rot on the vine before they can be merged.

Re: [Python-Dev] Appeal for reviews

2014-04-12 Thread Benjamin Peterson
On Sat, Apr 12, 2014, at 17:30, Stephen J. Turnbull wrote: I apologize for the tone. I need to go *right* now, and can't fix that. Really, I'm sympathetic and my goal is not just to defend python-dev, but to help you get the reviews your work deserves. Please read with that in mind. I don't