On Fri, Apr 23, 2010 at 19:47, Aurélien Campeas
<aurelien.camp...@logilab.fr> wrote:
> Le vendredi 23 avril 2010 à 19:41 +0200, Sandro Tosi a écrit :
>> On Wed, Mar 24, 2010 at 19:12, Julien Jehannet
>> <julien.jehan...@logilab.fr> wrote:
>> >> * Sylvain Thénault <sylvain.thena...@logilab.fr> [22-mar-2010 12:18]:
>> >> 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 introduced in 2.5.3-1~exp1 so it might be
>> >> > better to check python version than distutils one.
>> >>
>> >> Julien, could you do the proper fix in lgc'setup.py please ?
>> > It's fixed in repo but I'm thinking about a new way now.
>>
>> The fix (as released in 0.50.0) is bugged:
>>
>> --- logilab-common~/setup.py 2010-04-20 11:00:32.000000000 +0200
>> +++ logilab-common/setup.py 2010-04-23 19:37:02.697413120 +0200
>> @@ -158,7 +158,7 @@
>> try:
>> if not USE_SETUPTOOLS:
>> # install-layout option was introduced in 2.5.3-1~exp1
>> - if sys.versioninfo < (2, 5, 4):
>> + if sys.version_info < (2, 5, 4):
>> sys.argv.remove('--install-layout=deb')
>> print "W: remove '--install-layout=deb' option"
>> except:
>>
>> because:
>>
>> $ python -c "import sys ; print sys.version_info"
>> (2, 5, 5, 'final', 0)
>
> I may be a bit tired but I fail to see a bug here.
look it better and you'll see versioninfo vs. version_info :)
The problem is not in the idea, but in the spelling.
Regards,
--
Sandro Tosi (aka morph, morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi
_______________________________________________
Python-Projects mailing list
Python-Projects@lists.logilab.org
http://lists.logilab.org/mailman/listinfo/python-projects
