<br><br><div class="gmail_quote">On Tue, Jan 25, 2011 at 1:53 PM, <span dir="ltr"><<a href="mailto:exarkun@twistedmatrix.com">exarkun@twistedmatrix.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 07:23 pm, <a href="mailto:kevin.horn@gmail.com">kevin.horn@gmail.com</a> wrote:<br>
>On Tue, Jan 25, 2011 at 11:54 AM, Jason J. W. Williams <<br>
><a href="mailto:jasonjwwilliams@gmail.com">jasonjwwilliams@gmail.com</a>> wrote:<br>
>>Hi Y'all,<br>
>Howdy, Jason!<br>
>>Certain servers (notably Hacker News) break HTTPClient because they<br>
>>use LF<br>
>>instead of CRLF to terminate header lines. I've uploaded a patch with<br>
>>tests<br>
>>to ticket 4814 (merged into ticket 2842).<br>
>><br>
>>I'd appreciate a review and some discussion about integrating it.<br>
>><br>
>>-J<br>
>I'm not really in a position to do a review right now, but since:<br>
><br>
>1) 4814 is effectively a duplicate of 2842<br>
>2) 4814 has a clear problem decription<br>
>3) 4814 has a patch<br>
>4) 2842 is old, and confusing<br>
><br>
>Can we go ahead and close 2842 in favor of 4814? We have too many old<br>
>tickets floating around...<br>
<br>
</div>#4814 is already closed. </blockquote><div><br>Ah. Somehow I thought it was still open.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
If the #2842 description is confusing, then<br>
feel free to update it to be more clear. :)<br></blockquote><div><br>Well, I can update the summary, but not the description, which is the really bad part.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
We don't close old tickets as duplicates of new tickets, though. That<br>
doesn't make sense, and it's annoying to the original reporter/cc list<br>
as well.<br></blockquote><div><br>This certainly makes sense in theory, but when the last comment from the nominal owner of the ticket is "I'm not sure what is supposed to be fixed now.", I'm not sure how helpful it is. Yes, I realize that you had responded to it, and clarified the situation. Just Saying. There's a LOT of mutant tickets out there, that don't mean what they say, or have completely changed in scope. And there are several which have been closed in favor of newer, clearer tickets over the years. I think we need to make the above policy obvious someplace, like in the developer docs. Maybe it's in there, I haven't checked, but I don't recall reading it, and I've read most of the documentation _many_ times.<br>
<br>In any case, I don't care _that_ much about _which_ one is closed as long as one is. Which appears to be the case.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">><br>
>(To be clear, I'm not asking Jason to close this ticket...I'm asking<br>
>for the<br>
>list's opinion)<br>
><br>
>Also 3833 is at least similar, if not another duplicate. It would be<br>
>nice<br>
>if it could be worked on and/or closed along with this issue.<br>
<br>
</div>Similar, certainly. But it's for the other HTTP client implementation.<br>
There won't be any code in common in the resolution.<br>
<br>
I certainly agree that if it's worth fixing for one it's worth fixing<br>
for the other, though, and I hope someone will put #3833 up for review<br>
soon. :)<br>
<br>
Jean-Paul<br>
<div><div></div><div class="h5"><br>
_______________________________________________<br>
Twisted-Python mailing list<br>
<a href="mailto:Twisted-Python@twistedmatrix.com">Twisted-Python@twistedmatrix.com</a><br>
<a href="http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python" target="_blank">http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python</a><br>
</div></div></blockquote></div><br>