[libvirt] thunderbird bug workaround (was Re: [PATCH libvirt] domain: add implicit USB controller)

[Paolo Bonzini]

On 02/08/2012 02:28 PM, Jiri Denemark wrote:
> On Wed, Feb 08, 2012 at 07:37:19 -0500, Cole Robinson wrote:
>> On 02/08/2012 02:26 AM, Osier Yang wrote:
>>>> diff --git a/tests/define-dev-segfault b/tests/define-dev-segfault
>>>> index 3feff46..0a0532b 100755
>>>> --- a/tests/define-dev-segfault
>>>> +++ b/tests/define-dev-segfault
>>>> @@ -36,6 +36,7 @@ cat<<\EOF>  D.xml || fail=1
>>>>     <on_crash>destroy</on_crash>
>>>>     <devices>
>>>>       <emulator>/usr/bin/qemu-kvm</emulator>
>>>> +<controller type='usb' index='0'/>
>>>>       <serial type='pty'>
>>>>         <target port='0'/>
>>>>       </serial>
>>
>> But the spacing here is weird... is that my mail client or the actual patch?
>> That <controller> should be indented. Other tests are similarly changed.
>
> I guess that's the famous thunderbird bug (or is that actually a feature
> now?).

I have no idea, but today I got fed up enough that I decided to work 
around it.  I placed an extension at 
http://people.redhat.com/pbonzini/format-flawed.tar.gz -- you're welcome 
to expand it in "~/.thunderbird/<profile name>/extensions" and test it.

Since the buggy code is in C++, it just disables format=flowed when 
replying to non-format=flowed messages.  It's ugly as hell, but it does 
the job.

Paolo