ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | [4.2] join ulysses@ | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | Ulysses Apokin <aoipkn> | ||||||
| Component: | join | Assignee: | Gleb F-Malinovskiy <glebfm> | ||||||
| Status: | ASSIGNED --- | QA Contact: | Andrey Cherepanov <cas> | ||||||
| Severity: | normal | ||||||||
| Priority: | P5 | CC: | glebfm, grenka, ldv, rider, zerg | ||||||
| Version: | unspecified | ||||||||
| Hardware: | x86_64 | ||||||||
| OS: | Linux | ||||||||
| Attachments: |
|
||||||||
|
Description
Ulysses Apokin
2024-10-22 14:48:37 MSK
Created attachment 17040 [details]
Публичная часть SSH-ключа
Менторство подтверждаю. Прошу выдать гитовницу! Ментор есть, ключи в порядке. ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3. Прошу выдать доступ к сборочнице. ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6. Кандидат собрал очень большое количество пакетов и мне кажется, что уловил основные тенденции сборки по альт. Рекомендую его для дальнейших проверок рецензентам. Призван рецензент (rider@) для независимой оценки готовности кандидата. T/J/S -> 4.2. https://git.altlinux.org/tasks/archive/done/_379/388339/logs/events.2.3.log В пакетах opensnitch лучше исправить ошибки с неупакованными каталогами. Как раз вышла новая версия. Присылайте на review обновление. В git мне не очень понравилось то, что тарболл делается всего дерева. Лучше делать тарболлы отдельно для вендоринга, отдельно для апстрима (из тэга). Так нагляднее видно в src.rpm ваши изменения. ---- В пакете libphosphor-logging https://packages.altlinux.org/ru/tasks/382793/ вы не использовали SharedLibsPolicy - этот пакет надо переделать, пока на него не появились зависимости. В этом же пакете надо улучшить description, сейчас он дублирует summary и очень слабый. И в нём же надо попробовать включить тесты. ---- В пакете conky https://packages.altlinux.org/ru/tasks/382376/ проделана большая работа, но URL в specfile остался недействительный (идёт редирект). Такое лучше всегда проверять и исправлять при каждом изменении пакета. И в этом же пакете апстрим сделал достаточно много тестов, которые лучше выполнить в секции %check ---- https://packages.altlinux.org/ru/tasks/381934/ пакет libgovarnam не сделан в соответствии с shared libs policy, это надо обязательно исправить. Так-же надо включить тесты (это замечание касается всех пакетов). https://bugzilla.altlinux.org/54897 - не надо ждать реакции апстрима, если у нас пакет сломан надо чинить. Ну и у вас в пакете, который использует эту библиотеку почему-то отключен PkgConfig. Это криво, предлагаю данный патч убрать и исправить работу c pkgconfig. Итого из того что я посмотрел - кандидат проделал большую работу но пока не готов к самостоятельной работе в репозитории. Менторы (cas и grenka) многие вещи апрувили без достаточной проверки. Предлагаю кандидату попробовать исправить все озвученные проблемы, в том числе в тех пакетах которые я не посморел (очевидно что недопонимание необходимости использования SharedLibsPolicy прослеживается везде). Присылайте новые задания на ревью уже мне (можно в телеграм). Антон, отправлял вам таски на почту. Сегодня продублировал письма. (In reply to Ulysses Apokin from comment #11) > Антон, отправлял вам таски на почту. > Сегодня продублировал письма. Ответил по почте, спасибо. https://git.altlinux.org/tasks/401479/gears/500/git?p=git;a=commitdiff;h=549b25ffb89e3fd6ee71af280a52834224338048 %{name}_%soversion-devel -- это черезчур. Обсуждение происходит в https://bugzilla.altlinux.org/56619 Eigen3 это библиотека шаблонов. Если происходят обратно-несовместимые изменения, и мы хотим плавной миграции, то стоит рассмотреть возможность сборки нескольких версий -devel пакетов. В данном случае не надо городить огород а просто надо исправить всех клиентов (Ответ для Anton Farygin на комментарий #15) > просто надо исправить всех клиентов Не ломая совместимость с p11. Возможно, сперва провести в p11 подготовку. |
