Last modified: 2014-05-11 14:23:02 UTC
family.versionnumber now uses a regex: R = re.compile(r"(\d+).(\d+)") M = R.search(self.version(code)) and then returns the following: return 1000 * int(M.group(1)) + int(M.group(2)) - 1000 so 1.10 is 10, 1.21 is 21, and 2.15 is 1015 (?!). We should probably either use "from distutils.version import StrictVersion" or parse the version in a (major, minor, sub) tuple.
I propose LooseVersion as LV which enables comparing mw Version like "1.23wmf17" directly e.g. >>> LV("1.23wmf1") < LV("1.23wmf2") < LV("1.24") < LV("1.24wmf1") < LV("1.24wmf10") < LV("2.1") True
Change 132741 had a related patch set uploaded by Xqt: (bug 65156) use LooseVersion to compare mw versions https://gerrit.wikimedia.org/r/132741
Change 132741 merged by jenkins-bot: (bug 65156) use LooseVersion to compare mw versions https://gerrit.wikimedia.org/r/132741