Re: starting to dive into python package bugs

2013-07-17 Thread Stéphane Blondon
2013/7/16 Thomas Kluyver : > I would be inclined *not* to use BaseException for this - the intention is > that 'except Exception:' should catch all normal exceptions, and only > KeyboardInterrupt and SystemExit are outside that. I don't know the > specifics of the string exceptions you're updating,

Re: starting to dive into python package bugs

2013-07-16 Thread Thomas Kluyver
On 16 July 2013 22:25, Stéphane Blondon wrote: > > I checked the string exceptions in the code and they are not catched > > (see shell commands used at this end of this message). > > So I plan to wrap the string with an exception (Exception ou > > TypeError). To me, the errors raised are not Type

Re: starting to dive into python package bugs

2013-07-16 Thread Stéphane Blondon
Hello everyone, one week passed without any remark so I suppose what I said was Ok. 2013/7/6 Stéphane Blondon : > I checked the string exceptions in the code and they are not catched > (see shell commands used at this end of this message). > So I plan to wrap the string with an exception (Except

Re: starting to dive into python package bugs

2013-07-06 Thread Stéphane Blondon
Thanks for your replies Barry and Thomas! According to apt-rdepends, there are no packages depending on python-forgetsql: $ apt-rdepends --reverse python-forgetsql Reading package lists... Done Building dependency tree Reading state information... Done python-forgetsql So it will simpler to fix!

Re: starting to dive into python package bugs

2013-07-03 Thread Thomas Kluyver
On 3 July 2013 19:36, Barry Warsaw wrote: > The reason is that if some code is trying to: > > except 'error message' > > this will fail if the raise site is changed. > In fact it will already fail - recent Python 2 versions throw a TypeError if you attempt to raise a bare string (from at lea

Re: starting to dive into python package bugs

2013-07-03 Thread Barry Warsaw
On Jul 02, 2013, at 09:42 PM, Stéphane Blondon wrote: >I found in 'http://udd.debian.org/cgi-bin/python_bugs.cgi' that the >'Python string exceptions' are listed several times and I think the >base to fix it is easy: >replacing "raise 'error message' " by "raise Exception('error >message')" should

starting to dive into python package bugs

2013-07-02 Thread Stéphane Blondon
Hello everyone, I found in 'http://udd.debian.org/cgi-bin/python_bugs.cgi' that the 'Python string exceptions' are listed several times and I think the base to fix it is easy: replacing "raise 'error message' " by "raise Exception('error message')" should fix the problem. I guess the fix will be m