18493 2009-01-12 22:57:19 +0300 Проблема со сравнением версий содержащих буквы 2012-12-22 19:26:21 +0400 1 1 4 Development Sisyphus librpm unstable all Linux ASSIGNED P2 normal --- 1 kas placeholder arseny dottedmag erthad glebfm imz ldv led n3npq placeholder vt qa-sisyphus oldest_to_newest 84333 0 3212 kas 2009-01-12 22:57:19 +0300 Created attachment 3212 fix for rpmvercmp Словил забавный баг(фичу?) в rpmvercmp. Есть две версии "1.a" и "1a". rpmvercmp считает их равными! rpm сравнивает эти версии следующим образом (упрощено): 1. Отбрасывает из начала обоих строк символы не являющиеся буквами или цифрами (!isalnum) в начале строк. В данном случае в обоих строках таких символов нет. 2. Выбирает из обоих строк максимальное количество количество цифр и сравнивает, как числа. 1 == 1, т.е. на данном этапе версии равны. 3. Далее сравнивает оставшиеся части версий: ".a" и "a". 4. Тут цикл начинается с начала. Опять отбрасываются все символы не являющиеся буквами и цифрами. Остаётся "a" и "a". Они, очевидно, являются равными. Я считаю такое поведение ошибочным. Отбрасывать символы не являющиеся буквами или цифрами следует только, если в обоих строках они есть. Как вы считаете? 85846 1 ldv 2009-02-14 02:26:34 +0300 Вроде бы все считают, что надо этот патч приложить. 85852 2 led 2009-02-14 04:07:35 +0300 (В ответ на комментарий №1) > Вроде бы все считают, что надо этот патч приложить. Исправлять нужно, но ИМХО не этим патчем. 85857 3 kas 2009-02-14 08:43:37 +0300 А что с этим патчем не так? 86034 4 at 2009-02-17 09:07:12 +0300 А какая версия должна быть больше, 1a или 1.a? Нет ответа => лучше не патчить. 86037 5 dottedmag 2009-02-17 09:31:17 +0300 Очевидно, что 1a > 1.a. 128059 6 n3npq 2012-01-03 03:34:59 +0400 tracked at https://bugs.launchpad.net/rpm/+bug/911038 3212 2009-01-12 22:57:19 +0300 2009-01-12 22:57:19 +0300 fix for rpmvercmp diff text/plain 582 kas ZGlmZiAtLWdpdCBhL3JwbWRiL3JwbXZlcmNtcC5jIGIvcnBtZGIvcnBtdmVyY21wLmMKaW5kZXgg NWFjNTRkZC4uMTUzOWFlNSAxMDA2NDQKLS0tIGEvcnBtZGIvcnBtdmVyY21wLmMKKysrIGIvcnBt ZGIvcnBtdmVyY21wLmMKQEAgLTM0LDggKzM0LDEwIEBAIGludCBycG12ZXJjbXAoY29uc3QgY2hh ciAqIGEsIGNvbnN0IGNoYXIgKiBiKQogCiAgICAgLyogbG9vcCB0aHJvdWdoIGVhY2ggdmVyc2lv biBzZWdtZW50IG9mIHN0cjEgYW5kIHN0cjIgYW5kIGNvbXBhcmUgdGhlbSAqLwogICAgIHdoaWxl ICgqb25lICYmICp0d28pIHsKLQl3aGlsZSAoKm9uZSAmJiAheGlzYWxudW0oKm9uZSkpIG9uZSsr OwotCXdoaWxlICgqdHdvICYmICF4aXNhbG51bSgqdHdvKSkgdHdvKys7CisJaWYgKCgqb25lICYm ICF4aXNhbG51bSgqb25lKSkgJiYgKCp0d28gJiYgIXhpc2FsbnVtKCp0d28pKSkgeworCSAgICB3 aGlsZSAoKm9uZSAmJiAheGlzYWxudW0oKm9uZSkpIG9uZSsrOworCSAgICB3aGlsZSAoKnR3byAm JiAheGlzYWxudW0oKnR3bykpIHR3bysrOworCX0KIAogCWlmICggISpvbmUgJiYgISp0d28gKQog CQlyZXR1cm4gMDsK