Page 1 of 1

Bug in comparison of VersionSplits -> deluge/common.py

Posted: Wed Apr 14, 2010 5:49 pm
by anardil
When evaluating the version comparison of libtorrent version 0.14.10.0 with the required one by deluge 0.14.9.0, the string comparison evaluates to false for:

0.14.10.0 >= 0.14.9.0

This leads to a risen Error at _libtorrent.py:53
The attached patch fixes it by comparing the integer converted subparts of the versionstring.

Maybe you can fix this in a later version.

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Wed Apr 14, 2010 5:52 pm
by Cas
what version of deluge is this for because i am sure this was fixed in 1.2.3

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Thu Apr 15, 2010 5:04 pm
by anardil
I also checked that by checking out the latest version via git and there still was the problem in common.py

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Thu Apr 15, 2010 5:08 pm
by anardil
Oh by the way the link http://dev.deluge-torrent.org/browser to browse the source is not working.

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Thu Apr 15, 2010 6:38 pm
by Cas
I have tested with 1.2.3, there appears to be no problem here.

Code: Select all

>>> from deluge.common import VersionSplit
>>> VersionSplit("0.14.10.0") > VersionSplit("0.14.9.0")
True
anardil wrote:Oh by the way the link http://dev.deluge-torrent.org/browser to browse the source is not working.
Yep its a trac issue but you can still browse source here: http://git.deluge-torrent.org/

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Thu Apr 15, 2010 7:01 pm
by anardil
Damn I looked in the wrong place when inspecting the current common.py

As you say, its fixed in the initializer, sorry for the inconvenience.

I'm gonna upgrade to 1.2.3 as soon as possible (cool version number by the way ;-)

Re: Bug in comparison of VersionSplits -> deluge/common.py

Posted: Fri Apr 16, 2010 7:23 am
by Cas
i thought that might have happened, thats for the input thouigh, it is apreciated :)