Bug 52444

Created attachment 17396 [details]
SSH Public key

Ключи выглядят ОК, для продолжения процедуры потребуется подтверждение ментора.

Принимаю кандидата.

кандидат готов начать вступление

glebfm@, похоже пора уже мне на 2.3, больше месяца на 2.0 нахожусь.

ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.

Кандидат готов собирать пакеты. Прошу предоставить доступ к gyle.alt.

ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.
Адрес подписан на devel@.

T/J/S -> 3.6.

Кандидат готов собирать пакеты в сизиф.
Прошу призвать рецензента.

https://git.altlinux.org/tasks/391697/

jfyi, для сборки модуля под ядро не нужно пересобирать ядро в этом же задании.

ps. Да и стоит ли форкать модули рази изменения .gear/km-karch

(Ответ для Vitaly Chikunov на комментарий #10)
> https://git.altlinux.org/tasks/391697/
> 
> jfyi, для сборки модуля под ядро не нужно пересобирать ядро в этом же
> задании.

Пересобирал из-за "error: incompatible gcc/plugin versions".

Призван рецензент (rider@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.

Всё посмотрел, к тому что собрано вопросов не возникло.
Но пакеты в основном на go, нет C/C++ библиотек - непонятно освоил ли кандидат SharedLibsPolicy (один из самых сложных для понимания новичками моментов).

rider@, прошу проверить таск, который связан с Shared Libs Policy: https://packages.altlinux.org/ru/tasks/403806/

В задании ошибка, нельзя допускать переименования libxml2 -> libxml2_2 для legacy библиотеки. Лучше оставить как было (libxml2) и она умрёт со временем сама, когда будут пересобраны все клиенты.

если клиенты все пересобираются в задании, то legacy библиотека уже не нужна

И ещё лучше не перегружать спекфайл дополнительными условиями, т.к. сompat библиотека делается крайне редко, то проще и чище удалить из specfile для неё всё лишнее и сделать её отдельной веткой.

а с этим условиям compat спек получается перегружен, дополнительные проверки. Плюс если встретятся вложенные условия то какие-то парсеры specfile могут поломаться (на примере сизифного add_changelog)

Требование перести legacy библиотеку в другой пакет было моё. Так эе было моим требованием добавить переменные compat. Поэтому Максим все сделал правильно.
Пакет сопровождать дальше мне, поэтому и требования выставляю я.

(Ответ для Alexey Shabalin на комментарий #18)
> Требование перести legacy библиотеку в другой пакет было моё. Так эе было
> моим требованием добавить переменные compat. Поэтому Максим все сделал
> правильно.
> Пакет сопровождать дальше мне, поэтому и требования выставляю я.

а зачем ты сразу учишь плохому ? ;)

моя задача посмотреть что сделал кандидат и я реагирую не на то, что ты попросил а то что он сделал, в данном случае основная проблема (libxml2 -> libxml2_2) - ошибка, которая почти однозначно приведёт к проблемам с обновлением пакета (Obsoletes вижу, но лучше всё-равно не рисковать). В данном случае прекрасный повод такой ошибки избежать, за счёт смены soname у пакета.

я считаю наоборот, что твои рекомендации приведут к проблемам, и ты учишь плохому.

и что делать будем ?

(In reply to Alexey Shabalin from comment #20)
> я считаю наоборот, что твои рекомендации приведут к проблемам,

Уточни, пожалуйста, к каким именно проблемам.

> и ты учишь плохому.

Обоснуй, пожалуйста.

Видимо я был не прав. Отзываю свои предыдущие требования.
Максим, сделай:
- не надо переименовывать libxml2 -> libxml2_2 для legacy библиотеки
- не перегружай спекфайл дополнительными условиями.

(Ответ для Alexey Shabalin на комментарий #23)
> Видимо я был не прав. Отзываю свои предыдущие требования.
> Максим, сделай:
> - не надо переименовывать libxml2 -> libxml2_2 для legacy библиотеки
> - не перегружай спекфайл дополнительными условиями.

rider@, я обновил таск. Прошу проверить.