<div class="gmail_quote">On Mon, Oct 5, 2009 at 1:49 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;">
<div class="gmail_quote"><div class="im">On Mon, Oct 5, 2009 at 1:32 PM, Glyph Lefkowitz <span dir="ltr">&lt;<a href="mailto:glyph@twistedmatrix.com" target="_blank">glyph@twistedmatrix.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;">

<div class="gmail_quote"><div>On Mon, Oct 5, 2009 at 12:34 PM, Kevin Horn <span dir="ltr">&lt;<a href="mailto:kevin.horn@gmail.com" target="_blank">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><div><br>Excellent!<br> </div><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><div><br>Your analysis on the ticket looks fairly thorough.<br>

</div></div></blockquote></div><div><br>Thanks!  Though, really I just went over the changesets that therve submitted, and checked them against exarkun&#39;s suggestions.<br> </div><div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div class="gmail_quote"><div>
 </div><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><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>
<br></blockquote><div> </div></div><div>That&#39;s kind of what I figured.  <br><br>From my previous review of the current status, it looks pretty well done to me, but  I&#39;ll start taking a look at it in more detail, and see what I can come up with.  Might need some more tests, etc.<br>

<br>Kevin &quot;Someone&quot; Horn </div></div></blockquote><div><br>FYI, this ticket back up for review...<br><br>Kevin Horn <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

</blockquote></div><br>