25927 2011-07-19 13:46:07 +0400 Ошибка в формировании переменных версии пакета 2011-08-20 10:36:02 +0400 1 1 4 Development Sisyphus cmake unstable all Linux CLOSED NOTABUG P3 major --- 1 serpiph dubrsl lav nenderus zerg qa-sisyphus oldest_to_newest 123309 0 5015 serpiph 2011-07-19 13:46:07 +0400 Created attachment 5015 Правка исходного кода Согласно файла modules/readme.txt, при отсутствии указания версии переменная XXX_FIND_VERSION должна отсутствовать, переменные XXX_FIND_VERSION_MAJOR, XXX_FIND_VERSION_MINOR, XXX_FIND_VERSION_PATCH и XXX_FIND_VERSION_TWEAK, XXX_FIND_VERSION_COUNT. На данный момент переменная XXX_FIND_VERSION есть всегда и она определена, переменные XXX_FIND_VERSION_MAJOR, XXX_FIND_VERSION_MINOR, XXX_FIND_VERSION_PATCH и XXX_FIND_VERSION_TWEAK при отсутствии указания версии устанавливаются в случайное значение. При этом исполнение сборки модулей непредсказуемо от падения при сборке до неправильно собранных файлов. 123337 1 dubrsl 2011-07-20 11:14:44 +0400 вот тут может сломали или около http://git.altlinux.org/people/dubrsl/packages/?p=cmake.git;a=commitdiff;h=9fc7ea4c620e849a719ea23ea6faff17f1d67ee6 123341 2 serpiph 2011-07-20 14:53:14 +0400 Прошу прощения, в процессе написания дрогнула рука. Должно быть: "Согласно файла Modules/readme.txt, при отсутствии указания версии переменная XXX_FIND_VERSION должна отсутствовать, переменные XXX_FIND_VERSION_MAJOR, XXX_FIND_VERSION_MINOR, XXX_FIND_VERSION_PATCH и XXX_FIND_VERSION_TWEAK, XXX_FIND_VERSION_COUNT должны отсутствовать." Может, мой патч и неверен. В общем, в kde4libs-devel в файле /usr/share/kde4/apps/cmake/modules/FindQt4.cmake есть проверка переменной Qt4_FIND_VERSION и в случае наличия данной переменной выставляются другие переменные. Однако при вызове этого файла через 'find_package(Qt4)' переменная Qt4_FIND_VERSION создаётся и задаётся параметром QT_MIN_VERSION, а переменные Qt4_FIND_VERSION_* или созданы и пусты, или не созданы вообще. В результате падение при сборке программ, использующих KDE4 и, соответственно, файл из kde4libs-devel, так как в в файле FindKDE4.cmake из kde4libe-devel как раз стоит вызов find_package без версии, но с заранее заданным параметром QT_MIN_VERSION. Глубже разобраться не получается. P.S. Ага! В файле Modules/FindQt4.cmake из исходников cmake есть такая запись: #support old QT_MIN_VERSION if set, but not if version is supplied by find_package() IF(NOT Qt4_FIND_VERSION AND QT_MIN_VERSION) SET(Qt4_FIND_VERSION ${QT_MIN_VERSION}) ENDIF(NOT Qt4_FIND_VERSION AND QT_MIN_VERSION) Случаем, не она портит всю малину? В таком случае здесь должны определяться ещё и переменные Qt4_FIND_VERSION_*, чего нет. 123363 3 serpiph 2011-07-21 13:15:29 +0400 Как только я исправил файл Modules/FindQt4.cmake из cmake-modules, так сразу сборка заработала. 123364 4 serpiph 2011-07-21 13:16:48 +0400 Но вопрос к исходному коду (неинициализации переменных) всё равно никуда не делся. 123431 5 serpiph 2011-07-25 00:16:40 +0400 Вопрос к коду остался. 123602 6 dubrsl 2011-08-03 22:45:46 +0400 Вот мнение девелопера http://public.kitware.com/Bug/view.php?id=12352#c27160 124304 7 serpiph 2011-08-20 10:36:02 +0400 Понятно. Тогда вопрос к коду закрываю. 5015 2011-07-19 13:46:07 +0400 2011-07-19 13:46:07 +0400 Правка исходного кода cmake-2.8.5.patch text/plain 1100 serpiph ZGlmZiAtTmF1ciBjbWFrZS0yLjguNS5vcmlnL1NvdXJjZS9jbUZpbmRQYWNrYWdlQ29tbWFuZC5j eHggY21ha2UtMi44LjUvU291cmNlL2NtRmluZFBhY2thZ2VDb21tYW5kLmN4eAotLS0gY21ha2Ut Mi44LjUub3JpZy9Tb3VyY2UvY21GaW5kUGFja2FnZUNvbW1hbmQuY3h4CTIwMTEtMDctMDggMTU6 MTI6NTAuMDAwMDAwMDAwICswNDAwCisrKyBjbWFrZS0yLjguNS9Tb3VyY2UvY21GaW5kUGFja2Fn ZUNvbW1hbmQuY3h4CTIwMTEtMDctMTkgMTM6Mzk6MDEuNjQwMDE1NzAzICswNDAwCkBAIC0xNzMx LDEwICsxNzMxLDEwIEBACiAKICAgICAgIC8vIFRyeSB0byBwYXJzZSB0aGUgdmVyc2lvbiBudW1i ZXIgYW5kIHN0b3JlIHRoZSByZXN1bHRzIHRoYXQgd2VyZQogICAgICAgLy8gc3VjY2Vzc2Z1bGx5 IHBhcnNlZC4KLSAgICAgIHVuc2lnbmVkIGludCBwYXJzZWRfbWFqb3I7Ci0gICAgICB1bnNpZ25l ZCBpbnQgcGFyc2VkX21pbm9yOwotICAgICAgdW5zaWduZWQgaW50IHBhcnNlZF9wYXRjaDsKLSAg ICAgIHVuc2lnbmVkIGludCBwYXJzZWRfdHdlYWs7CisgICAgICB1bnNpZ25lZCBpbnQgcGFyc2Vk X21ham9yID0gMDsKKyAgICAgIHVuc2lnbmVkIGludCBwYXJzZWRfbWlub3IgPSAwOworICAgICAg dW5zaWduZWQgaW50IHBhcnNlZF9wYXRjaCA9IDA7CisgICAgICB1bnNpZ25lZCBpbnQgcGFyc2Vk X3R3ZWFrID0gMDsKICAgICAgIHRoaXMtPlZlcnNpb25Gb3VuZENvdW50ID0KICAgICAgICAgc3Nj YW5mKHRoaXMtPlZlcnNpb25Gb3VuZC5jX3N0cigpLCAiJXUuJXUuJXUuJXUiLAogICAgICAgICAg ICAgICAgJnBhcnNlZF9tYWpvciwgJnBhcnNlZF9taW5vciwKQEAgLTE3NTEsNiArMTc1MSw3IEBA CiAgICAgfQogCiAgIHJlc3VsdF92ZXJzaW9uID0gdGhpcy0+TWFrZWZpbGUtPkdldFNhZmVEZWZp bml0aW9uKCJQQUNLQUdFX1ZFUlNJT04iKTsKKyAgaWYoIXRoaXMtPlZlcnNpb25Gb3VuZENvdW50 KSB0aGlzLT5NYWtlZmlsZS0+UmVtb3ZlRGVmaW5pdGlvbigiUEFDS0FHRV9GSU5EX1ZFUlNJT04i KTsKICAgaWYgKHJlc3VsdF92ZXJzaW9uLmVtcHR5KCkpCiAgICAgewogICAgIHJlc3VsdF92ZXJz aW9uID0gInVua25vd24iOwo=