Dear Mattia,

Thanks for picking up the issue on such short notice. I saw the issue
and patch on launchpad, I'll be following and awaiting that issue as
well to see how and when the problem is resolved.

Mattia Rizzolo wrote:
>> I have implemented the following patch locally to work around the
>> problem:
>>
>> --- /usr/share/inkscape/extensions/scour.inkscape.py~
>> +++ /usr/share/inkscape/extensions/scour.inkscape.py
>> @@ -6,3 +6,6 @@
>>      import scour
>> -    from scour.scour import scourString
>> +    try:
>> +        from scour.scour import scourString
>> +    except Exception as e:
> If anything, this should catch inly ImportError over all of Exception
> (besides, there is no need to keep the exception info in the 'e'
> variable here).

You are obviously correct, there is no need to catch every exception or
to store it in `e`. I guess I was lazy in implementing the fix, and
duplicated the `except` statement from two lines below (I implemented
the change when I quickly needed a fix, submitted the report about an
hour later).

> Thank you again for your bug!

I am just a user, I'm sure I don't deserve a lot of credit. :-) Thanks
again for the effort and the quick action.

Kind regards,

Tim.

Reply via email to