[Python-projects] epydoc warnings when building logilab-common doc

2010-03-22 Thread Sandro Tosi
Hello, there are quite a bunch of warnings when building logilab-common doc (you can see them in the Debian build-log attached). Given they are quite boring when you try to find "real" problems when preparing the package :) , it would be super if someone can take the task to fix them. Thanks in a

Re: [Python-projects] pylint #4581 proposed fix

2010-03-22 Thread Donovan Lee Wanhoy
Hi, Here is a new diff file with the changes. Donovan Lee Wanhoy On Mon, Mar 22, 2010 at 12:30 PM, Emile Anclin wrote: > On Thursday 18 March 2010 06:33:01 Donovan Lee Wanhoy wrote: > > Hi, > > > > Is there a reason to make it optional, rather than have it enabled at > > all > > > > > times? >

[Python-projects] ANN : pylint bug day 2nd edition

2010-03-22 Thread Alexandre Fayolle
Hi everyone we'll hold the next `pylint bugs day`_ on april 16th 2010 (friday). If some of you want to come and work with us in our `Paris office`_, you'll be much welcome. Else you can still join us on jabber / irc: * jabber: chat room pub...@jabber.logilab.org * irc: #public on irc://irc.lo

[Python-projects] Fix for Pylint #9018, when using defining-attr-method, method order matters

2010-03-22 Thread Scott Pilkey
Hello, I came up with this fix a little while ago, but haven't gotten a chance to submit my proposed fix until now. It's very straight-forward. In the existing code, you only check the first item in any node list, when the entire list actually needs to be checked to see if the defining metho

Re: [Python-projects] pylint #4581 proposed fix

2010-03-22 Thread Emile Anclin
On Thursday 18 March 2010 06:33:01 Donovan Lee Wanhoy wrote: > Hi, > > Is there a reason to make it optional, rather than have it enabled at > all > > > times? > > I assumed that the bug ticket said to make it optional. Attached is the > same code except there is no option check. > Hope this one is

Re: [Python-projects] Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Sylvain Thénault
On 22 mars 11:58, Sandro Tosi wrote: > > I rather think the current test about distutils version is inadequate. > > Because '--install-layout=dev' doesn't seem to exist for python 2.5.2-3 > > (lenny) but *does* for python 2.5.4-1 (distutils version still the same). > > Yep, you're right: it was in

Re: [Python-projects] Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Sandro Tosi
Hi Julien, On Mon, Mar 22, 2010 at 11:32, Julien Jehannet wrote: > Hello Sandro, > >> * Sandro Tosi [22-mar-2010 08:38]: >> Hi Sylvain, >> >> On Mon, Mar 22, 2010 at 08:25, Sylvain Thénault >> wrote: >> > On 20 mars 10:18, Sandro Tosi wrote: >> >> I fear it's still wrong. distutils 2.5.1 suppor

[Python-projects] Re: Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Julien Jehannet
Hello Sandro, > * Sandro Tosi [22-mar-2010 08:38]: > Hi Sylvain, > > On Mon, Mar 22, 2010 at 08:25, Sylvain Thénault > wrote: > > On 20 mars 10:18, Sandro Tosi wrote: > >> I fear it's still wrong. distutils 2.5.1 supports --install-layout=deb > >> but the check: > >> > >> In [3]: tuple([int(x)

Re: [Python-projects] Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Sylvain Thénault
On 22 mars 08:37, Sandro Tosi wrote: > > If you tell me we shoud use --install-layout=dev with python < 2.5.1 and not > > <= 2.5.1, changing the comparison into strict comparison should definitly > > do the trick and I'll do that. > > Well, what I say is that distutils version 2.5.1 supports > --i

Re: [Python-projects] Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Sandro Tosi
Hi Sylvain, On Mon, Mar 22, 2010 at 08:25, Sylvain Thénault wrote: > On 20 mars 10:18, Sandro Tosi wrote: >> I fear it's still wrong. distutils 2.5.1 supports --install-layout=deb >> but the check: >> >> In [3]: tuple([int(x) for x in distutils_version.split('.')]) <= (2, 5, 1) >> Out[3]: True >>

Re: [Python-projects] pylint #18860: warn on assert( a, b ?) patch

2010-03-22 Thread Sylvain Thénault
On 21 mars 01:07, Fletcher Johnson wrote: > Did you want it to warn on all types that are always true? > I'm looking at the types here in '6.1 Truth Value Testing'-> > http://docs.python.org/library/stdtypes.html and it says that "All > other values are considered true — so objects of many types ar

Re: [Python-projects] Check for distutils version for --install-layout=deb is broken

2010-03-22 Thread Sylvain Thénault
On 20 mars 10:18, Sandro Tosi wrote: > Hi Sylvain, Hi Sandro, > I fear it's still wrong. distutils 2.5.1 supports --install-layout=deb > but the check: > > In [3]: tuple([int(x) for x in distutils_version.split('.')]) <= (2, 5, 1) > Out[3]: True > > that causes again to drop that option. You c

Re: [Python-projects] A fix for Pylint #8776, inline message supression

2010-03-22 Thread Sylvain Thénault
On 21 mars 11:33, Alex Margarit wrote: > Hey, Hi, sorry, I've been missing some time last week to review pylint patches. Hopefuly this week will be better :) > I was wondering if anyone had the time to take a look at the patch I > sent for Pylint ticket #8776. Do you think my solution is approp