[merge] Fix trailing whitespace bug #49182

[Robey Pointer]

On 22 Jun 2006, at 12:50, John Arbash Meinel wrote:

> Aaron Bentley wrote:
>> John Arbash Meinel wrote:
>>>> I've gotten bit by this several times, and it seems other people  
>>>> have as
>>>> well. So I went ahead and wrote this patch which tests bundles  
>>>> that have
>>>> their trailing whitespace messed with, and fixes the parser.
>>
>> Great.  +1.
>>
>>>> The selftests pass, though for some reason I'm getting  
>>>> tracebacks in the
>>>> SFTP Server threads. 'Broken pipe' mostly. But I'm getting those in
>>>> plain 'bzr.dev', so it shouldn't be anything with my patch.
>>
>> Those seem to have come in with Robey's recent SFTP timeout fix.
>
> Yeah, that's what I guessed. I wouldn't have +1'd it if I had seen  
> that
> happening. It seems that on my FC4 machine, it doesn't happen. But on
> Ubuntu Dapper it does.

It's happening on my mac now, too.  Turns out it's a bug in paramiko  
that's fixed in the trunk but not in a release (yet).  Attached is a  
patch against bzr to absorb that exception.  Actually as an  
experiment I'm going to attach it as a bundle.

robey


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: broken-pipe-bundle.txt
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20060622/8029203c/attachment.txt