Bug 35053
| Summary: | [done] join nir@ | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | Igor Chudov <nir> | ||||||||
| Component: | join | Assignee: | Dmitry V. Levin <ldv> | ||||||||
| Status: | CLOSED FIXED | QA Contact: | Andrey Cherepanov <cas> | ||||||||
| Severity: | normal | ||||||||||
| Priority: | P3 | CC: | gkot, glebfm, iv, ldv, mike, nickel, sin, vseleznv, zerg | ||||||||
| Version: | unspecified | ||||||||||
| Hardware: | all | ||||||||||
| OS: | Linux | ||||||||||
| URL: | http://www.altlinux.org/Team/Join/Secretary | ||||||||||
| Attachments: |
|
||||||||||
Created attachment 7598 [details]
SSH pubkey
С моей стороны принято. Нужно подобрать пакет и освоить сборку. Я так понимаю, что сборка освоена. Нужно предоставить доступ в git, чтобы можно было посмотреть результаты. Адрес для пересылки создан, ssh ключ на gitery.alt и gyle.alt зарегистрирован. T/J/S -> 3.0. Переделал пакет так, чтобы использовались макросы RPM: ssh://git@github.com/NIR-Ginko/ficl.git ветка `alt` Заглянул -- есть пара нюансов, которые прямщас не смертельны, но вообще-то добавят предсказуемой мороки в дальнейшем. 1) Version: 4.2.0rc1 > 4.2.0, соответственно не получится обновить пакет до релизного без добавления строчки Serial: 1 или Epoch: 1 (см. https://www.altlinux.org/Spec#Epoch); обычный обход -- Version: 4.2.0 Release: alt0.1 (по крайней мере до поддержки tilda versions в ALT RPM) 2) тот же тег прибит в .gear/rules, в то время как gear немного умеет разбирать spec-файл и делать макроподстановки вроде @name@, @version@, @release@. Ещё я бы убрал BuildRequires: gcc (заведомо входит в сборочную среду),
все BuildPreReq заменил на BuildRequires(pre) по текущей моде,
ну и для полной красоты заменил %{_bindir} на %_bindir
(или прогнал по спеку cleanup_spec из пакета rpm-utils).
(In reply to comment #7) > все BuildPreReq заменил на BuildRequires(pre) по текущей моде, При чём тут мода? Они используются для разного: Пакеты записанные в теге BuildRequires(pre) устанавливает hasher на самой ранней стадии (ещё до обработки spec-а rpm-build-ом). А BuildPreReq эквивалентен BuildRequires, но удобен в spec-ах, к которым применяется buildreq(1) потому что buildreq(1) заменяет теги BuildRequires на вычисленный список пакетов. Т.е. можно, например, записать в BuildPreReq пакеты, которые buildreq(1) не видит. http://git.altlinux.org/people/nir/packages/?p=ficl.git;a=summary - перенёс код из GitHub. Вроде бы учёл все поправки. Прошу проверить. Добрый день. Есть более интересная задача: http://git.altlinux.org/people/nir/packages/?p=syslinux6.git;a=summary Обновил syslinux с 4.04 до версии 6.04. Очень нужен был lpxelinux.0 и бинари для UEFI boot. Проверьте лучше этот пакет, а то его уже надо бы отправить на сборку и тестирование. Пакеты подготовлены, нужно переходить к T/J/S -> 4.0. По syslinux стоит координироваться с zerg@ (ему и другим релиз-менеджерам важен патч gfxboot); другие интересанты -- legion@ и gremlin@. Но это лучше открыть отдельный баг по обновлению syslinux, нежели обсуждать здесь. (В ответ на комментарий №12) > По syslinux стоит координироваться с zerg@ (ему и другим релиз-менеджерам важен > патч gfxboot); другие интересанты -- legion@ и gremlin@. Но это лучше открыть > отдельный баг по обновлению syslinux, нежели обсуждать здесь. + nickel@ В syslinux нет еще релиза 6.04. https://repo.or.cz/syslinux.git Какой из "pre" вы собираете? Uefi бинарники проверяли на работоспособность? http://git.altlinux.org/people/nickel/packages/?p=syslinux.git;a=shortlog;h=refs/heads/redo_from_scratch_to_pre2_git.bf6db5 (В ответ на комментарий №10) [...] > > Обновил syslinux с 4.04 до версии 6.04. Очень нужен был lpxelinux.0 и бинари > для UEFI boot. > если очень нужно можно взять из чернового задания: 236472 > Проверьте лучше этот пакет, а то его уже надо бы отправить на сборку и > тестирование. Вас ждут сюрпризы, как минимум этот: https://bugzilla.altlinux.org/show_bug.cgi?id=34481#c19 (В ответ на комментарий №12) > По syslinux стоит координироваться с zerg@ Я уже сообщал, что достаточно не затрагивать пакет syslinux. Пакет alt-gpgkeys обновлён. T/J/S -> 4.0. Хочу протолкнуть пакет в таске 263779 в sisyphus. (In reply to Igor Chudov from comment #17) > Хочу протолкнуть пакет в таске 263779 в sisyphus. Думаю, что в данной ситуации разумно попросить о review и approve мейнтейнера обновляемого пакета. $ ssh girar task show --brief 263779 id=263779 locked=no shared=no fail_early=yes test_only=yes repo=sisyphus owner=nir state=TESTED try=1 iter=1 age=0w build_time=2020-Dec-22/15:09:35 100:gpupdate.git=0.8.2-alt1 $ ssh girar acl sisyphus gpupdate show gpupdate sin @everybody $ ssh girar task approve 263779 100 task approve: Go ahead and type the text of approval Newest release from upstream developer $ ssh girar task run --commit 264114 girar-check-perms: access to gpupdate DENIED for nir: does not belong to maintainers list yet check-subtask-perms: #100: gpupdate: needs approvals from members of @maint and @tester groups task run: a reason must be specified for 'p9' repository Try `task run --help' for more information. Чего-то мне не хватает. (In reply to Igor Chudov from comment #20) > $ ssh girar task run --commit 264114 [...] > task run: a reason must be specified for 'p9' repository > Try `task run --help' for more information. > > Чего-то мне не хватает. Попросите, пожалуйста, вашего ментора помочь вам понять то, что я процитировал. (Ответ для Dmitry V. Levin на комментарий #21) ... > > Чего-то мне не хватает. > > Попросите, пожалуйста, вашего ментора помочь вам понять то, что я > процитировал. Это я попросил Игоря приложить лог, а вот комментарий к нему следовало написать другой. Вопрос был в том, что требуется собрать пакет в p9, сборка пакетов уже освоена, процедура join'а не завершена. Что для этого нужно сделать? Сначала Игорь собрал свою разработку в пакет ficl, затем syslinux6 оказался на острие в плане того, что за него уже брались другие и была проблема бинарей в сборочнице. А теперь собирает admc и gpupdate. Со своей стороны предлагаю завершить эту задачу поправленным вариантом первого пакета - ficl. (In reply to Evgeny Sinelnikov from comment #22) > (Ответ для Dmitry V. Levin на комментарий #21) > ... > > > task run: a reason must be specified for 'p9' repository > > > Try `task run --help' for more information. > > > > > > Чего-то мне не хватает. > > > > Попросите, пожалуйста, вашего ментора помочь вам понять то, что я > > процитировал. > > Это я попросил Игоря приложить лог, а вот комментарий к нему следовало > написать другой. Вопрос был в том, что требуется собрать пакет в p9, сборка > пакетов уже освоена, процедура join'а не завершена. Что для этого нужно > сделать? У нас разные вопросы, это нормально. Меня больше всего сейчас интересует, как Игорь понимает ту диагностику от сборочницы, которую процитировал сперва он, а потом я. Я не посмотрел сообщение об ошибке и сделал глупость. Перезапущу. Касательно ficl - не уверен, что его стоит собирать в Sisyphus. Оно очень специфичное, abandonware. Плюс, 4.2.0rc1 это мой собственный форк, в котором я просто сделал обработку shebang. Не вижу смысла просто так его затаскивать в Sisyphus, так как это создание лишней нагрзуки на мейнтейнеров без реального приложения ПО. $ ssh girar task run -m "Increased_D-Bus_timeouts" --commit 264114 girar-check-perms: access to gpupdate DENIED for nir: does not belong to maintainers list yet check-subtask-perms: #100: gpupdate: needs approvals from members of @maint and @tester groups task #264114: try #3 is AWAITING, result will be emailed to nir@altlinux.org Я так понимаю, что я до сих по не попал в "maintainers list". Как решить данную часть вопроса? (In reply to Igor Chudov from comment #25) > $ ssh girar task run -m "Increased_D-Bus_timeouts" --commit 264114 > girar-check-perms: access to gpupdate DENIED for nir: does not belong to > maintainers list yet > check-subtask-perms: #100: gpupdate: needs approvals from members of @maint > and @tester groups > task #264114: try #3 is AWAITING, result will be emailed to nir@altlinux.org > > Я так понимаю, что я до сих по не попал в "maintainers list". Как решить > данную часть вопроса? Но вы же понимаете, что этот факт никак не влияет на судьбу вашего задания в p9, правда? (In reply to Dmitry V. Levin from comment #26) > (In reply to Igor Chudov from comment #25) > > $ ssh girar task run -m "Increased_D-Bus_timeouts" --commit 264114 > > girar-check-perms: access to gpupdate DENIED for nir: does not belong to > > maintainers list yet > > check-subtask-perms: #100: gpupdate: needs approvals from members of @maint > > and @tester groups > > task #264114: try #3 is AWAITING, result will be emailed to nir@altlinux.org > > > > Я так понимаю, что я до сих по не попал в "maintainers list". Как решить > > данную часть вопроса? > > Но вы же понимаете, что этот факт никак не влияет на судьбу вашего задания в > p9, правда? Да, понимаю. Я хочу завершить процедуру join и получить право отправлять пакеты на сборку в sisyphus. (In reply to Igor Chudov from comment #27) > Да, понимаю. Я хочу завершить процедуру join и получить право отправлять > пакеты на сборку в sisyphus. Это понятно, но ... (In reply to Evgeny Sinelnikov from comment #22) > Это я попросил Игоря приложить лог, а вот комментарий к нему следовало > написать другой. Вопрос был в том, что требуется собрать пакет в p9, сборка > пакетов уже освоена, процедура join'а не завершена. Что для этого нужно > сделать? ... мне пока не удалось понять позицию ментора по этому вопросу. (In reply to Dmitry V. Levin from comment #28) > ... мне пока не удалось понять позицию ментора по этому вопросу. 2sin: ping (Ответ для Dmitry V. Levin на комментарий #29) > (In reply to Dmitry V. Levin from comment #28) > > ... мне пока не удалось понять позицию ментора по этому вопросу. > > 2sin: ping Прошу прощения, я долго выходил из рекурсии по данному вопросу. Если я правильно понимаю, то вопрос ставится так: "Какова позиция ментора по вопросу о том, завершена ли процедура join'а? И что требуется для её завершения, если по не завершена?" С моей стороны уже прозвучало соображение, что "сборка пакетов уже освоена". И я, со своей стороны, рассчитываю от секретаря на ответ: "Достаточно, ли текущих сборок, которые ранее были представлены, включая текущие сборки gpudate, для завершения процедуры принятия nir@ в ALT?" Я считаю, что технически вполне достаточно. (In reply to Evgeny Sinelnikov from comment #30) > Я считаю, что технически вполне достаточно. T/J/S -> 4.0 Призван ещё один ментор для независимой оценки готовности кандидата. По собранным в Сизиф пакетам у меня серьёзных замечаний нет. Могу высказать следующие: * При манипуляции с файлами (например, копировании) лучше сохранять метаданные файлов, например, использовать команду cp -a вместо простого cp или cp -r. * В gpupdate явно прописано: Requires: python3-module-rpm Requires: python3-module-dbus автогенератор зависимостей их не определяет? * Крайне желательно отделять изменения непосредственно в коде и в спеках/специфично gear'овских файлах в разные коммиты. Так гораздо проще откатывать, черрипикать и проводить другие изменения с проектом. При учтении этих замечаний, я думаю, что кандидат готов. Как разработчик Игорь субъективно давно уж готов. Добрый день. (In reply to Vladimir D. Seleznev from comment #33) > По собранным в Сизиф пакетам у меня серьёзных замечаний нет. Могу высказать > следующие: > > * При манипуляции с файлами (например, копировании) лучше сохранять > метаданные файлов, например, использовать команду cp -a вместо простого cp > или cp -r. > Учтём в следующих релизах. > * В gpupdate явно прописано: > > Requires: python3-module-rpm > Requires: python3-module-dbus > > автогенератор зависимостей их не определяет? > Судя по комментариям коммитов из git blame - не определяет. > * Крайне желательно отделять изменения непосредственно в коде и в > спеках/специфично gear'овских файлах в разные коммиты. Так гораздо проще > откатывать, черрипикать и проводить другие изменения с проектом. > У нас разработка продукта и релизы достаточно сильно кореллируют с изменениями specfile. Это же не сторонний проект, который мы "затаскиваем" в дистрибутив, а "родной". Так что в изменения specfile прекрасно вписываются в процесс разработки. > При учтении этих замечаний, я думаю, что кандидат готов. (In reply to Igor Chudov from comment #35) ... > > > * В gpupdate явно прописано: > > > > Requires: python3-module-rpm > > Requires: python3-module-dbus > > > > автогенератор зависимостей их не определяет? > > > > Судя по комментариям коммитов из git blame - не определяет. Определённо не определяет: $ rpm -qR gpupdate | grep rpm python3-module-rpm rpmlib(PayloadIsLzma) $ rpm -qR gpupdate | grep dbus python3-module-dbus $ git grep dbus | grep :import gpoa/frontend/appliers/systemd.py:import dbus gpoa/util/dbus.py:import dbus $ git grep rpm | grep :import gpoa/util/rpm.py:import rpm Пересобрал - проверил: $ gear-rpm -ba --commit ... Записан: /home/sin/build/rpm/SRPMS/gpupdate-0.8.2-alt1.src.rpm Записан: /home/sin/build/rpm/RPMS/noarch/gpupdate-0.8.2-alt1.noarch.rpm $ rpm -qpR /home/sin/build/rpm/RPMS/noarch/gpupdate-0.8.2-alt1.noarch.rpm | grep dbus $ rpm -qpR /home/sin/build/rpm/RPMS/noarch/gpupdate-0.8.2-alt1.noarch.rpm | grep rpm rpmlib(PayloadIsLzma) $ git diff | cat diff --git a/gpupdate.spec b/gpupdate.spec index b51c73d..9a76288 100644 --- a/gpupdate.spec +++ b/gpupdate.spec @@ -14,8 +14,8 @@ Requires: control BuildRequires: rpm-build-python3 BuildRequires: python-tools-i18n -Requires: python3-module-rpm -Requires: python3-module-dbus +#Requires: python3-module-rpm +#Requires: python3-module-dbus Requires: oddjob-%name >= 0.2.0 Requires: libnss-role >= 0.5.0 Requires: local-policy >= 0.4.0 К слову о том, на что ещё имеет смысл обращать внимание мантейнеру пакетов: $ grep warning: beehive/logs/Sisyphus/x86_64/latest/success/gpupdate-0.8.2-alt1 |sort -u warning: File listed twice: /usr/lib/python3/site-packages/gpoa/gpoa warning: File listed twice: /usr/lib/python3/site-packages/gpoa/gpupdate warning: File listed twice: /usr/lib/python3/site-packages/gpoa/gpupdate-setup warning: Macro %python3_sitelibdir not found warning: absolute symlink: /usr/src/tmp/gpupdate-buildroot/usr/bin/gpupdate -> /usr/lib/python3/site-packages/gpoa/gpupdate warning: absolute symlink: /usr/src/tmp/gpupdate-buildroot/usr/sbin/gpoa -> /usr/lib/python3/site-packages/gpoa/gpoa warning: absolute symlink: /usr/src/tmp/gpupdate-buildroot/usr/sbin/gpupdate-setup -> /usr/lib/python3/site-packages/gpoa/gpupdate-setup (In reply to Vladimir D. Seleznev from comment #33) > При учтении этих замечаний, я думаю, что кандидат готов. Продлите, пожалуйста, ключ: pub 4096R/0CAE7AAC 2018-06-18 [expired: 2020-06-17] uid Igor Chudov <nir@altlinux.org> (Ответ для Dmitry V. Levin на комментарий #37) > warning: absolute symlink: /usr/src/tmp/gpupdate-buildroot/usr/bin/gpupdate > -> /usr/lib/python3/site-packages/gpoa/gpupdate ln -sr в помощь :-) Created attachment 9249 [details]
GPG key extended to 2022-06-23
GPG public key with expiration date set to 2022.06.23. I would prefer to generate the new key at some moment in future but extending lifetime a bit is okay for now.
(In reply to Dmitry V. Levin from comment #39) > Продлите, пожалуйста, ключ: > pub 4096R/0CAE7AAC 2018-06-18 [expired: 2020-06-17] > uid Igor Chudov <nir@altlinux.org> Добрый день. Продлил, приложил к задаче. Адрес подписан на список рассылки devel@. Пользователь добавлен в группу мантейнеров. Желаю удачного мантейнерства! |
Description
Igor Chudov 2018-06-18 05:22:39 MSK