40869 2021-09-06 23:00:46 +0300 проблемы упаковки 2025-06-03 16:45:17 +0300 1 1 4 Development Sisyphus kitty unstable all Linux CLOSED FIXED P5 enhancement --- 1 mike egori darktemplar egori qa-sisyphus oldest_to_newest 202403 0 9641 mike 2021-09-06 23:00:46 +0300 Created attachment 9641 kitty.spec.diff Я тут заглянул в kitty.spec и огорчился -- там целый ряд странностей (минимум одна из которых наведена апстримом, но всё-таки тогда её бы и хорошо починить да предложить). 1) как будто тестируется _предыдущая_ сборка, ну или имеем лишний self-BR (бишь пакет надо зачем-то бутстрапить, чтобы разместить в репозитории, где его нет): # For tests BuildRequires: kitty Проверил -- зависимость для %check излишня (возможно, обломок экспериментов). 2) сборка выполняется в секции %install (%build попросту нет) [!] 3) в секциях %install и %check игнорируется макронабор из rpm-build-python3, всё сделано врукопашную 3a) впрочем, setup.py там какой-то самопальный, что ли (как порой неавтокраповые "configure" люди пишут): setup.py: error: argument action: invalid choice: 'install' (choose from 'build', 'test', 'linux-package', 'kitty.app', 'linux-freeze', 'macos-freeze', 'build-launcher', 'build-frozen-launcher', 'clean', 'export-ci-bundles') -- и вот здесь хорошо бы сделать цель install да предложить её в апстрим: мол, дальше с линуксовым пакетом сами разберёмся, просто не ломайте схему (от этой цели требуется уметь обрабатывать или игнорировать опции --skip-build, --root=<путь>, --force). Т.е. хорошо бы прийти к возможности свести эту часть спека к чему-то вроде %build %python3_build_debug %install %python3_install --update-check-interval=0 ...и если перетащить из rpm-build-intro в rpm-build-python3 макросы %python3_test/%python3_check -- получится и %check %python3_test Сейчас можно сделать +%build +%python3_build_debug -- кажется, при этом вторая пересборка во время %install проскочила быстро. На всякий прилагаю spec.diff относительно 0.21.2-alt1, который получился при проверке сборки на e2kv4; в нём ещё затычка для лишнего оптимизма с -Werror (--ignore-compiler-warnings получилось бы применить лишь без комментариев): [8/98] Compiling kitty/child-monitor.c ... lcc: "/usr/include/features.h", line 382: warning #1105: #warning directive: _FORTIFY_SOURCE requires compiling with optimization (-O) # warning _FORTIFY_SOURCE requires compiling with optimization (-O) ^ lcc: "kitty/screen.c", line 1437: error #186: pointless comparison of unsigned integer with zero [-Werror=type-limits] if (top <= self->cursor->y && self->cursor->y <= bottom) { ^ lcc: "kitty/screen.c", line 1463: error #186: pointless comparison of unsigned integer with zero [-Werror=type-limits] if (top <= self->cursor->y && self->cursor->y <= bottom) { ^ 2 errors detected in the compilation of "kitty/screen.c". 202414 1 egori 2021-09-07 14:43:57 +0300 Исправил пункты 1 и 2 в версии 0.23.1-alt1 [1] > -- и вот здесь хорошо бы сделать цель install да предложить её в апстрим: > мол, дальше с линуксовым пакетом сами разберёмся, просто не ломайте схему > (от этой цели требуется уметь обрабатывать или игнорировать опции > --skip-build, --root=<путь>, --force). Сейчас попробую сделать что-нибудь с setup.py [1]: http://git.altlinux.org/gears/k/kitty.git?p=kitty.git;a=commit;h=8f1627e3d84ba251449083b4de6a786a1673a77d 266370 2 egori 2025-06-03 16:45:17 +0300 Думаю на данный момент kitty упаковывается достаточно хорошо, на сколько позволяют особенности апстрима. 9641 2021-09-06 23:00:46 +0300 2021-09-06 23:00:46 +0300 kitty.spec.diff kitty.spec.diff text/plain 1089 mike ZGlmZiAtLWdpdCBhLy5nZWFyL2tpdHR5LnNwZWMgYi8uZ2Vhci9raXR0eS5zcGVjCmluZGV4IGEy NDFlMzRiLi42ZDY2MDAxNCAxMDA2NDQKLS0tIGEvLmdlYXIva2l0dHkuc3BlYworKysgYi8uZ2Vh ci9raXR0eS5zcGVjCkBAIC0yNyw3ICsyNyw2IEBAIEJ1aWxkUmVxdWlyZXM6IG5jdXJzZXMKIAog IyBGb3IgdGVzdHMKIEJ1aWxkUmVxdWlyZXM6IC9wcm9jCi1CdWlsZFJlcXVpcmVzOiBraXR0eQog QnVpbGRSZXF1aXJlczogZm9udHMtdHRmLWdudS1mcmVlZm9udC1tb25vCiAKICVhZGRfcHl0aG9u M19wYXRoICVfbGliZXhlY2Rpci8lbmFtZQpAQCAtNzksMTAgKzc4LDIwIEBAIFRoZSB0ZXJtaW5m byBmaWxlIGZvciBraXR0eQogJXNldHVwCiAKICMgQ2hhbmdpbmcgc2hlYmFuZ3MgdG8gcHl0aG9u MwotZmluZCAtdHlwZSBmIC1uYW1lICIqLnB5IiAtZXhlYyBzZWQgLWUgJ3N8L3Vzci9iaW4vZW52 IHB5dGhvbjN8JXtfX3B5dGhvbjN9fGcnICBcCi0gICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAtZSAnc3wvdXNyL2Jpbi9lbnYgcHl0aG9ufCV7X19weXRob24zfXxnJyAgIFwKK2Zp bmQgLXR5cGUgZiAtbmFtZSAiKi5weSIgLWV4ZWMgc2VkIC1lICdzfC91c3IvYmluL2VudiBweXRo b24zfCVfX3B5dGhvbjN8ZycgXAorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg LWUgJ3N8L3Vzci9iaW4vZW52IHB5dGhvbnwlX19weXRob24zfGcnICBcCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAtaSAie30iIFw7CiAKKyVpZmFyY2ggJWUyaworIyBmdGJm cyB3b3JrYXJvdW5kIHdpdGggbGNjIDEuMjUuMTc6CisjIGtpdHR5L3NjcmVlbi5jOjE0MzcsMTQ2 MzogcG9pbnRsZXNzIGNvbXBhcmlzb24gb2YgdW5zaWduZWQgaW50ZWdlciB3aXRoIHplcm8KK3Nl ZCAtaSAicy8tV2Vycm9yLy9nIiBzZXR1cC5weQorJWVuZGlmCisKKyMgVE9ETzogcHJvcGVyICdp bnN0YWxsJyB0YXJnZXQKKyNidWlsZAorI3B5dGhvbjNfYnVpbGRfZGVidWcKKwogJWluc3RhbGwK IG1rZGlyIC1wICVidWlsZHJvb3QlX3ByZWZpeAogcHl0aG9uMyBzZXR1cC5weSBsaW51eC1wYWNr YWdlIFwK