| Summary: | [done] join k0tran@ | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | Ilya Sorochan <sorochanilyav> | ||||||
| Component: | join | Assignee: | Gleb F-Malinovskiy <glebfm> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | Andrey Cherepanov <cas> | ||||||
| Severity: | normal | ||||||||
| Priority: | P5 | CC: | glebfm, grenka, iv, ldv, rider, zerg | ||||||
| Version: | unspecified | ||||||||
| Hardware: | x86_64 | ||||||||
| OS: | Linux | ||||||||
| Attachments: |
|
||||||||
|
Description
Ilya Sorochan
2024-07-30 14:21:59 MSK
Created attachment 16513 [details]
Публичный gpg ключ
Менторство подтверждаю. Спасибо iv@ несколько моих коммитов уже в git.alt: - comsic-comp https://git.altlinux.org/gears/c/cosmic-comp.git?p=cosmic-comp.git;a=commitdiff;h=439e96cb230d7b1af4b125ca6877d60e0e0a075d - python3-module-flask-babel https://git.altlinux.org/gears/p/python3-module-flask-babel.git?p=python3-module-flask-babel.git;a=commitdiff;h=9b0f598d54dce0b499101667ed54c5413a7bc2f4 - hping3 https://git.altlinux.org/gears/h/hping3.git?a=commitdiff;h=45fadce2cb66d36aeb097de90cb2f953a09e7990 Все коммиты фиксят работу на loongarch64 Прогресс на текущий момент: сумел вроде как неплохо запаковать (на мой взгляд) четыре разных проекта. oh-my-zsh, менеджер плагинов для zsh. Написан на шелле https://github.com/k0tran/oh-my-zsh-sisyphus typst-lsp, LSP для Typst. Написан на Rust https://github.com/k0tran/typst-lsp-sisyphus2 zvm, менеджер версий для zig. Написан на Go https://github.com/k0tran/zvm-sisyphus zls, LSP для Zig. Написан на Zig https://github.com/k0tran/zls-sisyphus Репозитории временные, позже снесу Прошу секретаря проверить ssh и gpg ключи (пункт 1.2 join'а) Ментор есть, ключи в порядке. T/J/S -> 1.3. Прошу выдать гитовницу. Подопечный готов собирать пакеты. ping ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3. Прошу выдать гитовницу. *сборочницу. ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6. Little update для тех кто следит 1. Собрал simdutf таска 359766 (/people/k0tran/packages/simdutf.git) 2. Собрал zls таска 359768 (/people/k0tran/packages/zls.git) Обе одобрены ментором, продолжаю собирать пакетики. (In reply to Ilya Sorochan from comment #13) > 1. Собрал simdutf таска 359766 (/people/k0tran/packages/simdutf.git) Я почти заглянул, мне кажется, что тут ещё есть вопросы. То, что библиотека с программой оказались в одном пакете это очень неаккуратно. Менее важно: 1 Такие Provides: излишни (apt умеет apt-get install /usr/bin/sdutf); 2. Есть подозрение, что /proc на самом деле нужен только для %check (т.е. скорее всего можно перенести под условие). 3. Ну и мелочь, которая на самом деле действительно важна только в контексте предыдущего пункта (но различие лучше тоже понимать). BuildRequires(pre): это зависимости, которые обрабатываются раньше вообще всех остальных и на них не влияют %if-ы в spec-файлах (они буквально sed-ом выделяются из spec-файла). Зависимости для монтирования /proc, /dev/pts и /dev/kvm не требуют обработки на такой ранней стадии и для них следует использовать BuildRequires: или BuildPreReq: (они на практике ничем не отличаются). В целом, всё остальное, что я увидел выглядит хорошо. Пользуясь случаем, что кандидат "щёлкает" и не отправляет заппрувленный таск, отозвал аппрув до исправления вышеуказанных претензий. (In reply to Gleb F-Malinovskiy from comment #14) > (In reply to Ilya Sorochan from comment #13) > > 1. Собрал simdutf таска 359766 (/people/k0tran/packages/simdutf.git) > Я почти заглянул, мне кажется, что тут ещё есть вопросы. То, что библиотека > с программой оказались в одном пакете это очень неаккуратно. Изначально паковал библиотеку, она будет нужна ladybird, если тот когда-нибудь выйдет из альфы и мы захотим его себе. Бинари упаковал просто потому что они есть и теоритически могут быть нужны. Игнорировать их было бы совсем некрасиво В остальном, Provides убрал, /proc передвинул внутрь %check (убрав (pre)), cпецифику BuildRequires(pre) при сборке gear/hasher нашел. Не стал делать отдельные релиз и таску, мне показалось в данном случае лучше такие мелочи поправить и не плодить сущностей. Соотвествено таска та же 359766 Вообще изначально /proc я взял отсюда https://www.altlinux.org/SampleSpecs/cmakeprogram Когда его увидел, подумал что возможно он нужен cmake для чего-то Approved. (In reply to Ilya Sorochan from comment #16) > Изначально паковал библиотеку, она будет нужна ladybird, если тот > когда-нибудь выйдет из альфы и мы захотим его себе. Бинари упаковал просто > потому что они есть и теоритически могут быть нужны. Они могут быть запакованы с библиотекой только если *библиотека* ими пользуется, но так бывает очень редко и обычно такие программы кладут куда-нибудь в /usr/lib64 или /usr/libexec . > Игнорировать их было бы совсем некрасиво. На самом деле скорее наоборот (как я вижу, в Fedora утилиты совсем не собирают -DSIMDUTF_TOOLS=OFF) . В любом случае, если паковать утилиты, то в отдельный пакет, именно поэтому я изначально стал писать и именно это не исправлено. (In reply to Gleb F-Malinovskiy from comment #19) > > Игнорировать их было бы совсем некрасиво. > На самом деле скорее наоборот (как я вижу, в Fedora утилиты совсем не > собирают -DSIMDUTF_TOOLS=OFF) . В любом случае, если паковать утилиты, то > в отдельный пакет, именно поэтому я изначально стал писать и именно это не > исправлено. Справедливо. Добавил их флаги. Еще не увидел /proc (поэтому его убрал) и дополнительно добавил оттуда ExcludeArch. Все собралось там же 359766 (In reply to Ilya Sorochan from comment #20) > Справедливо. Добавил их флаги. Еще не увидел /proc (поэтому его убрал) У них оно как-то иначе реализовано (скорее всего, /proc всегда есть). В любом случае без него всё в порядке. > дополнительно добавил оттуда ExcludeArch Для нас это чистый мусор, а на ревью я в этом месте попросил бы добавить комментарий, почему такой ExcludeArch сделан. В остальном, lgtm. Спасибо. Просьба увеличить квоту на гитовнице - почистил все старые репозитории и все равно не могу собрать ppsspp (собрать его необходимо что бы приложить патч для лонгарча) (In reply to Ilya Sorochan from comment #22) > Просьба увеличить квоту на гитовнице - почистил все старые репозитории и все > равно не могу собрать ppsspp (собрать его необходимо что бы приложить патч > для лонгарча) Отедльно оформил в багу #52083 По поводу https://packages.altlinux.org/en/tasks/369850/ : Нужно убрать портянку из версии и поместить(при необходимости) её в релиз. Потому, что rpmvercmp 0.1.git20250112.54af769-alt1 0.1.0 возвращает то же, что 0.1.git20250112.54af769-alt1 0.1.5 В остальном ок. Пересобрал в той же таске c 0.1-alt1.git54af769 https://packages.altlinux.org/en/tasks/369850/ Один момент, нужно что-то сделать с .gear/version.sh Готово, выпилил скрипт напрочь Регулярный апдейт для неравнодушных: - nemu - tui для qemu - нормально протестировать пока руки не дошли - zellij - современный tmux (офк на расте), сделал страничку на вики - qalculate-qt - qt интерфейс для qalculate - gef - gdb плагин для реверсеров/пвнеров, сделал страничку на вики - kakoune & kakoune-lsp - модальный редактор с системой выделение -> действие Что ожидает в тасках (обсуждаем с ментором): - toad - kde todo приложение - ждет zerg@ - vieb - модальный браузер - rbw - обновил поломанный пакет (https://github.com/doy/rbw/issues/233) - clifm - консольный файловый менеджер без tui Так же сабмичу патчи на loongarch64/riscv64 в чужие пакеты (Ответ для Ilya Sorochan на комментарий #28) > - qalculate-qt - qt интерфейс для qalculate Требует тщательной проверки исходников каждый раз по причине русофобства автора. Считаю, что кандидат освоил сборку пакетов с забандленными зависимостями, частенько делает ошибки по невнимательности, но кажется, что это не в моих силах исправить. Передаю в руки рецензента. Давно не писал. Все еще пакечу, все еще жду ревьювера. - rbw доехал до сизифа в починенном состоянии - vieb пользуюсь несколько месяцев, тогда хотел (но не получилось) запаковать еще вещей и отправить ментору пачкой. Сейчас запаковал новую версию, но там ошибки связанные с electron36 и glibc на aarch64 https://git.altlinux.org/tasks/386380/logs/events.2.1.log Уже написал grenka@ и glebfm@ но если кто-то еще может помочь - буду очень признателен. Браузер классный :D (может стоит открыть багу?) - clifm snk@ успел запаковать быстрее - toad в планах обновить и посмотреть, пока не успел - zig обновил и починил на aarch64 (https://bugzilla.altlinux.org/53447) - zls обновил - zellij обновил - libsimdutf обновил и переименовал (soversion поменялся) - helix похозяйничал в багах https://bugzilla.altlinux.org/show_bug.cgi?id=50968#c5 https://bugzilla.altlinux.org/show_bug.cgi?id=53053#c3 Также периодически влезаю в другие пакеты что бы добавить патчи на loongarch64/risv64 > Сейчас запаковал новую версию, но там ошибки связанные с electron36 и glibc на aarch64 > https://git.altlinux.org/tasks/386380/logs/events.2.1.log > electron36#36.4.0-alt1:sisyphus+386380.100.2.1@1749226104 /usr/lib/ld-linux-aarch64.so.1(GLIBC_2.17)(64bit) Такое случается, когда в пакет кладут собранный в неизвестных условиях скачаный с интернета бинарник. (In reply to Ivan A. Melnikov from comment #32) > > Сейчас запаковал новую версию, но там ошибки связанные с electron36 и glibc на aarch64 > > https://git.altlinux.org/tasks/386380/logs/events.2.1.log > > > electron36#36.4.0-alt1:sisyphus+386380.100.2.1@1749226104 /usr/lib/ld-linux-aarch64.so.1(GLIBC_2.17)(64bit) > > Такое случается, когда в пакет кладут собранный в неизвестных условиях > скачаный с интернета бинарник. Так и есть, делал по аналогии с electron23, 28 и 29 Призван рецензент (rider@) для независимой оценки готовности кандидата. T/J/S -> 4.2. https://packages.altlinux.org/ru/sisyphus/srpms/libsimdutf20/3204070663243892321 Это очень странное чтение sharedlibspolicy. При обновлении пакета имя src пакета менять нужно в крайнем случае и этот крайний случай не ваш. Просьба привести пакет в порядок - имя src пакета сделать как в апстриме, бинарный пакет должен соответствовать sharedlibspolicy. https://packages.altlinux.org/ru/sisyphus/maintainers/k0tran/watch давайте попробуем обновить свои пакеты, присылайте на review сборочные задания. Можно в телеграм. (In reply to Anton Farygin from comment #35) > https://packages.altlinux.org/ru/sisyphus/srpms/libsimdutf20/ > 3204070663243892321 > > Это очень странное чтение sharedlibspolicy. При обновлении пакета имя src > пакета менять нужно в крайнем случае и этот крайний случай не ваш. Просьба > привести пакет в порядок - имя src пакета сделать как в апстриме, бинарный > пакет должен соответствовать sharedlibspolicy. На вики сказано просто "имя пакета", src или bin не уточняется (https://www.altlinux.org/Shared_Libs_Policy): > Библиотека должна быть упакована в пакет, имя которого меняется при каждой ломке ABI. > Пакет должен иметь название libfoo%abiversion, где %abiversion является изменяемой частью. А также извините за нескромный вопрос: почему src имя пакета менять очень нежелательно? Насколько знаю везде используются имена бинарных пакетов - вот с ними понятно почему лучше не трогать. В любом случае пакет уже отобрал sbolshakov@ (сам только недавно узнал, но не возражаю) и переименовал в simdutf как в апстриме. Таски по обновлению пришлю в телеграм. (In reply to Anton Farygin from comment #36) > Можно в телеграм. Там платно xD Если вам удобнее в телеграме - мой хэндл @k0tran (такой же как и здесь) Еще просьба сообщить как вам отправлять таски. По одной или какой-нибудь пачкой из нескольких штук? как отправлять таски мне без разницы, в телеграм теперь доступ должен быть. Апдейт на текущий момент: - kakoune & kakoune-lsp обновлены - mandown & zellij обновлен - ghostty патч для рабочих xkb ремапов - palemoon актуализирован url - python3-module-crosshair-tool параллелизация тестов - procs обновлен - gpg-tui обновлен - stormlib собран - waylock обновлен - qalculate-qt обновлен (как и советовал zerg@ код смотрел) И еще пара мелких loongarch64 патчей там сям Есть желание завести 25 гигов тестов для stormlib. Не в гит, а локально удостоверится что все собрано правильно и нигде не спряталась бага. Пока с этим трудности. https://github.com/ladislav-zezula/StormLib/issues/394 Также от rider@ была поставлена задача починить/вернуть git-branchless. Фикс теперь в апстриме но по релизу конкретных дат не дали https://github.com/arxanas/git-branchless/issues/1585#issuecomment-3283095473 Предпринимал неудачную попытку бэкпортнуть, возможно попробую снова. Бэкпорт осложнен почти годовалой давностью релиза. Кандидат освоил основные подходы и готов к самостоятельной работе в составе ALT Linux Team. Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства! |