<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 6, 2013 at 12:26 PM, Antoine Pitrou <span dir="ltr"><<a href="mailto:solipsis@pitrou.net" target="_blank">solipsis@pitrou.net</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Hello,<br>
<br>
I would like to point out that we currently fail at handling GSoC<br>
projects and bringing them to completion.<br>
<br>
One cruel example is the set of PEP 3121 / PEP 384 refactorings done by<br>
Robin Schreiber:<br>
<a href="http://bugs.python.org/issue?%40columns=id%2Cactivity%2Ctitle%2Ccreator%2Cassignee%2Cstatus%2Ctype&%40sort=-activity&%40filter=status&%40action=searchid&ignore=file%3Acontent&%40search_text=pep+3121&submit=search&status=-1%2C1%2C3" target="_blank">http://bugs.python.org/issue?%40columns=id%2Cactivity%2Ctitle%2Ccreator%2Cassignee%2Cstatus%2Ctype&%40sort=-activity&%40filter=status&%40action=searchid&ignore=file%3Acontent&%40search_text=pep+3121&submit=search&status=-1%2C1%2C3</a><br>


<br>
Robin has produced many patches that seem to reach the stated goal<br>
(refactor C extension modules to take advantage of the latest PEPs<br>
about module initialization and extension types definition).<br>
Unfortunately, tackling both goals at the same time produces big<br>
patches with a lot of churn; and it is also not obvious the PEP 384<br>
refactoring is useful for the stdlib (while the PEP 3121 refactoring<br>
definitely is).<br>
<br>
What didn't produce an alarm during Robin's work is that GSoC work is<br>
done in private. Therefore, other core developers than the mentor don't<br>
get to give an advice early, as would happen with any normal proposal<br>
done publicly (on the mailing-list or on the bug tracker). It is also<br>
likely that the mentor gets overworked after the GSoC period is over,<br>
is unable to finalize the patch and push it, and other core devs have a<br>
hard time catching up on the work and don't know what the shortcomings<br>
are.<br></blockquote></div><br><br></div><div class="gmail_extra">I would like to point out something that stands out in this list of issues: such a method of producing dozens of patches simultaneously is extremely unwise, unless there's a crucial piece of history I'm missing. It is much more prudent to start with one or two exemplary modules, and if those fully pass code review, send out patches for others. The reason is obvious - code review may turn up problems or requests for change. Going backwards to modify 57 patches is not something anyone would want to do.<br>

<br></div><div class="gmail_extra">Eli<br><br></div></div>