<div class="gmail_quote">On Mon, Oct 5, 2009 at 12:34 PM, Kevin Horn <span dir="ltr">&lt;<a href="mailto:kevin.horn@gmail.com">kevin.horn@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I&#39;d like to see this ticket get moving again.</blockquote><div><br>Excellent!<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I&#39;ve gone over what&#39;s been done so far (and put my comments in the ticket), and it looks like everything is at least mostly ready to be merged into the Twisted trunk.<br></blockquote><div><br>Your analysis on the ticket looks fairly thorough.<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">So what else needs to be done to get this to happen?  I&#39;m happy to try and get it done, but a little direction would be really handy. ;)<font color="#888888"><br>
</font></blockquote><div><br>The workflow is pretty simple.<br><ol><li>author submits change<br></li><li>reviewer reviews change</li><li>someone responds to feedback</li><li>&quot;someone&quot; becomes &quot;author&quot;, repeat</li>
</ol>Right now we&#39;re stuck in the middle of step 3, but given that therve does not appear to be actively working on this branch, the path forward is clear.  <i>You</i> become &quot;someone&quot;, and submit a patch that finishes dealing with the issues that you have mentioned as not already taken care of by therve&#39;s subsequent changesets.  The best way to do this is to submit a patch against the branch, rather than against trunk.  Given such a patch, a twisted committer can then apply the patch to the branch and submit to buildbots (etc) as appropriate.  By submitting it against the branch, you make it easy for the reviewer to examine the changes you&#39;re submitting in isolation from the rest of the larger change, to see if they address the points that you&#39;ve brought up.<br>
<br>If the reviewer is happy with your changes, then the patch + branch will be immediately merged to trunk; of course, the next review might have a little more feedback for you to deal with ;).<br><br>Thanks for your interest!<br>
</div></div>