ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | [3.0] join usoltsevpv@ | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | Usoltsev Petr <usoltsevpv> | ||||||||
| Component: | join | Assignee: | Gleb F-Malinovskiy <glebfm> | ||||||||
| Status: | NEW --- | QA Contact: | Andrey Cherepanov <cas> | ||||||||
| Severity: | normal | ||||||||||
| Priority: | P5 | CC: | glebfm, ldv, liannnix, liannnix | ||||||||
| Version: | unspecified | ||||||||||
| Hardware: | x86_64 | ||||||||||
| OS: | Linux | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Usoltsev Petr
2024-12-05 07:33:15 MSK
Created attachment 17326 [details]
ssh ключ
Created attachment 17327 [details]
gpg ключ
ментор - liannnix@ Created attachment 17349 [details]
gpg ключ @altlinux
Псевдоним: usoltsevpv Адрес пересылки: usoltsev05@mail.ru Ментор: Андрей Лимачко <liannnix@altlinux.org> Цель: Научиться собирать пакеты Ключи в порядке, для продолжения нужен ментор. Завел задачу по исправлению сборки a2ps: https://bugzilla.altlinux.org/53318 Исправление опубликовал тут: https://github.com/ProhorP/a2ps (Ответ для Usoltsev Petr на комментарий #7) > Завел задачу по исправлению сборки a2ps: > https://bugzilla.altlinux.org/53318 > Исправление опубликовал тут: > https://github.com/ProhorP/a2ps Менторство подтверждаю. По a2ps: Думаю, что это не удачный пакет для первого опыта. В нём применяется устаревшая схема ведения репозитория. Сейчас у нас принято вести gear-репозиторий пакета с сохранением git-истории коммитов из upstream. К сожалению, у a2ps нет официального git-репозитория. Так что нужно понимать, в данном случае, сохранение такого стиля ведения репозитория - это вынужденная мера. Теперь замечания: 1. Не стоит складывать все в один коммит. Обновление исходников до новой версии - один коммит. Обновление патчей - другой коммит. Поднятие версии и обновление spec - третий коммит. 2. Если необходимо удалить какой-то патч, то лучше вынести это в отдельный коммит. 3. Не стоит оставлять в spec'е закоментированные строчки с уже не нужными инструкциями. Их удаление вполне неплохо видно в git-diff. 4. Зачем удалён подпакет liba2ps-devel? Комментарий в spec'е очень невнятный. К тому же такое лучше вынести в отдельный коммит с пояснением в commit message, по какой причине это было сделано. 5. Если уж подпакеты liba2ps и liba2ps-devel необходимо удалить, то и их описание тоже нужно удалять. 6. Почему тогда оставлен liba2ps-devel-static? 7. %dir /usr/share/locale. Так делать ну надо. Директория уже принадлежит glibc-locales. [liannnix@liannnix-nb a2ps]$ rpm -qf /usr/share/locale glibc-locales-2.40.0.69.8566822197-alt1.x86_64 Может, ещё что пропустил. Когда всё в кашу в одном коммите, то разбираться сложно. Переделал с учетом всех замечаний. Переопубликовал в прежнем месте (https://github.com/ProhorP/a2ps) (Ответ для Usoltsev Petr на комментарий #10) > https://github.com/ProhorP/a2ps-4.15.6 [liannnix@ninkear a2ps-4.15.6]$ git log --oneline -n 26 | cat d1ba101 4.15.6-alt1 909f3e2 remove a2ps-glibc-2.38-fix.patch f4c5f6c remove a2ps-4.14-debian-fix-format-security.patch b03a7a7 remove a2ps-4.14-debian-fix-bad-free.patch 9f3c0ca remove a2ps-4.14-gentoo-CVE-2014-0466.patch df9e810 update a2ps-alt-koi8.edf.patch 7e48e4f The patch is third-party and does not apply to a2ps 4.15.6 47eb544 The patch is third-party and does not apply to a2ps 4.15.6 40b229c Applied in a2ps 4.14 1e5b4fe Applied in a2ps 4.15.6 983f107 Applied in a2ps 4.14 c9e0f90 The patch is third-party and does not apply to a2ps 4.15.6 03dc1cd The patch is third-party and does not apply to a2ps 4.15.6 137c210 The patch is third-party and does not apply to a2ps 4.15.6 2b02b28 The patch is third-party and does not apply to a2ps 4.15.6 719c02f The patch is third-party and does not apply to a2ps 4.15.6 0b3b26f The patch is third-party and does not apply to a2ps 4.15.6 cb6d330 Applied in a2ps 4.15.6 33eb44a the patch is third-party and does not apply to a2ps 4.15.6 1d54ae2 update a2ps to 4.15.6 Такая история категорически не приемлема: 1. Комментарии к коммитам не отражают суть изменений. 2. Не ясно, почему одни патчи удалены, а другие нет. 3. Не понятно, была ли необходимость удалять все этим патчи, если в предыдущей итерации они были поправлены. https://bugzilla.altlinux.org/53318 4. Если необходимо удалить сразу много патчей, то не стоит заморачиваться с отдельными коммитами. 5. При этом, в заголовке комментария к изменению не стоит указывать имена файлов, а сам комментарий стоит сделать многострочным. Например: https://git.altlinux.org/gears/s/samba.git?p=samba.git;a=commit;h=69aae9da98d767fda37d98d2854bf660feb87e70. Историю изменений стоит переделать. Патчи, которые удалены, нужно вернуть, если они на самом деле имеют какую-то ценность. Из истории изменений и комментария в spec'е это должно быть понятно. А оно вообще работает? Проверялась ли работоспособность итоговой сборки? (Ответ для Andrey Limachko на комментарий #11) > (Ответ для Usoltsev Petr на комментарий #10) > > https://github.com/ProhorP/a2ps-4.15.6 > > [liannnix@ninkear a2ps-4.15.6]$ git log --oneline -n 26 | cat > d1ba101 4.15.6-alt1 > 909f3e2 remove a2ps-glibc-2.38-fix.patch > f4c5f6c remove a2ps-4.14-debian-fix-format-security.patch > b03a7a7 remove a2ps-4.14-debian-fix-bad-free.patch > 9f3c0ca remove a2ps-4.14-gentoo-CVE-2014-0466.patch > df9e810 update a2ps-alt-koi8.edf.patch > 7e48e4f The patch is third-party and does not apply to a2ps 4.15.6 > 47eb544 The patch is third-party and does not apply to a2ps 4.15.6 > 40b229c Applied in a2ps 4.14 > 1e5b4fe Applied in a2ps 4.15.6 > 983f107 Applied in a2ps 4.14 > c9e0f90 The patch is third-party and does not apply to a2ps 4.15.6 > 03dc1cd The patch is third-party and does not apply to a2ps 4.15.6 > 137c210 The patch is third-party and does not apply to a2ps 4.15.6 > 2b02b28 The patch is third-party and does not apply to a2ps 4.15.6 > 719c02f The patch is third-party and does not apply to a2ps 4.15.6 > 0b3b26f The patch is third-party and does not apply to a2ps 4.15.6 > cb6d330 Applied in a2ps 4.15.6 > 33eb44a the patch is third-party and does not apply to a2ps 4.15.6 > 1d54ae2 update a2ps to 4.15.6 > > Такая история категорически не приемлема: > 1. Комментарии к коммитам не отражают суть изменений. > 2. Не ясно, почему одни патчи удалены, а другие нет. > 3. Не понятно, была ли необходимость удалять все этим патчи, если в > предыдущей итерации они были поправлены. https://bugzilla.altlinux.org/53318 Все патчи кроме 2-х являются сторонними и не прикладываются к коду, т.к. кодовая база изменилась. Я сначала не понял этого и решил их всех по возможности обновить. Но потом понял что правильнее для новой версии a2ps скачать патчи по необходимости и приложить их. Осталось 2 патча наших: a2ps-4.13-alt-liba2ps_with_lm.patch a2ps-alt-koi8.edf.patch Первы > 4. Если необходимо удалить сразу много патчей, то не стоит заморачиваться с > отдельными коммитами. > 5. При этом, в заголовке комментария к изменению не стоит указывать имена > файлов, а сам комментарий стоит сделать многострочным. Например: > https://git.altlinux.org/gears/s/samba.git?p=samba.git;a=commit; > h=69aae9da98d767fda37d98d2854bf660feb87e70. > > Историю изменений стоит переделать. Патчи, которые удалены, нужно вернуть, > если они на самом деле имеют какую-то ценность. Из истории изменений и > комментария в spec'е это должно быть понятно. > > А оно вообще работает? Проверялась ли работоспособность итоговой сборки? Попытка установки частично удачная: [user@sisyphus x86_64]$ rpm -i a2ps-4.15.6-alt1.x86_64.rpm error: Failed dependencies: /bin/lp is needed by a2ps-4.15.6-alt1.x86_64 Причина в том что /bin/lp не принадлежит ни к одному пакету: ``` [user@sisyphus x86_64]$ rpm -qf /bin/lp file /bin/lp is not owned by any package [user@sisyphus x86_64]$ readlink -f /bin/lp /usr/bin/lp-cups [user@sisyphus x86_64]$ rpm -qf /usr/bin/lp-cups cups-2.4.12-alt1.x86_64 ``` Поэтому установил так: ``` rpm -i --nodeps a2ps-4.15.6-alt1.x86_64.rpm ``` После установки пакета проверил работу так: ``` echo "Hello, a2ps test!" > test.txt a2ps test.txt -o test.ps [test.txt (plain): 1 page on 1 sheet] [Total: 1 page on 1 sheet] saved into the file `test.ps' ``` Работает получается. (Ответ для Usoltsev Petr на комментарий #12) > (Ответ для Andrey Limachko на комментарий #11) > > (Ответ для Usoltsev Petr на комментарий #10) > > > https://github.com/ProhorP/a2ps-4.15.6 > > > > [liannnix@ninkear a2ps-4.15.6]$ git log --oneline -n 26 | cat > > d1ba101 4.15.6-alt1 > > 909f3e2 remove a2ps-glibc-2.38-fix.patch > > f4c5f6c remove a2ps-4.14-debian-fix-format-security.patch > > b03a7a7 remove a2ps-4.14-debian-fix-bad-free.patch > > 9f3c0ca remove a2ps-4.14-gentoo-CVE-2014-0466.patch > > df9e810 update a2ps-alt-koi8.edf.patch > > 7e48e4f The patch is third-party and does not apply to a2ps 4.15.6 > > 47eb544 The patch is third-party and does not apply to a2ps 4.15.6 > > 40b229c Applied in a2ps 4.14 > > 1e5b4fe Applied in a2ps 4.15.6 > > 983f107 Applied in a2ps 4.14 > > c9e0f90 The patch is third-party and does not apply to a2ps 4.15.6 > > 03dc1cd The patch is third-party and does not apply to a2ps 4.15.6 > > 137c210 The patch is third-party and does not apply to a2ps 4.15.6 > > 2b02b28 The patch is third-party and does not apply to a2ps 4.15.6 > > 719c02f The patch is third-party and does not apply to a2ps 4.15.6 > > 0b3b26f The patch is third-party and does not apply to a2ps 4.15.6 > > cb6d330 Applied in a2ps 4.15.6 > > 33eb44a the patch is third-party and does not apply to a2ps 4.15.6 > > 1d54ae2 update a2ps to 4.15.6 > > > > Такая история категорически не приемлема: > > 1. Комментарии к коммитам не отражают суть изменений. > > 2. Не ясно, почему одни патчи удалены, а другие нет. > > 3. Не понятно, была ли необходимость удалять все этим патчи, если в > > предыдущей итерации они были поправлены. https://bugzilla.altlinux.org/53318 > Все патчи кроме 2-х являются сторонними и не прикладываются к коду, т.к. > кодовая база изменилась. Я сначала не понял этого и решил их всех по > возможности обновить. Но потом понял что правильнее для новой версии a2ps > скачать патчи по необходимости и приложить их. Осталось 2 патча наших: > a2ps-4.13-alt-liba2ps_with_lm.patch > a2ps-alt-koi8.edf.patch Первый не нужен, т.к. upstream удалил создание библиотек. А второй я обновил и оставил только его. Добавить новые патчи как у Fedora планировал в будущем по требованию. > > 4. Если необходимо удалить сразу много патчей, то не стоит заморачиваться с > > отдельными коммитами. > > 5. При этом, в заголовке комментария к изменению не стоит указывать имена > > файлов, а сам комментарий стоит сделать многострочным. Например: > > https://git.altlinux.org/gears/s/samba.git?p=samba.git;a=commit; > > h=69aae9da98d767fda37d98d2854bf660feb87e70. > > > > Историю изменений стоит переделать. Патчи, которые удалены, нужно вернуть, > > если они на самом деле имеют какую-то ценность. Из истории изменений и > > комментария в spec'е это должно быть понятно. > > > > А оно вообще работает? Проверялась ли работоспособность итоговой сборки? > Попытка установки частично удачная: > [user@sisyphus x86_64]$ rpm -i a2ps-4.15.6-alt1.x86_64.rpm > error: Failed dependencies: > /bin/lp is needed by a2ps-4.15.6-alt1.x86_64 > > Причина в том что /bin/lp не принадлежит ни к одному пакету: > ``` > [user@sisyphus x86_64]$ rpm -qf /bin/lp > file /bin/lp is not owned by any package > [user@sisyphus x86_64]$ readlink -f /bin/lp > /usr/bin/lp-cups > [user@sisyphus x86_64]$ rpm -qf /usr/bin/lp-cups > cups-2.4.12-alt1.x86_64 > ``` > Поэтому установил так: > ``` > rpm -i --nodeps a2ps-4.15.6-alt1.x86_64.rpm > ``` > > После установки пакета проверил работу так: > ``` > echo "Hello, a2ps test!" > test.txt > a2ps test.txt -o test.ps > [test.txt (plain): 1 page on 1 sheet] > [Total: 1 page on 1 sheet] saved into the file `test.ps' > ``` > Работает получается. Переделал историю коммитов https://github.com/ProhorP/a2ps-4.15.6 Финальный коммит сделал через "gear-commit -a" Сделал обновление gettext до 0.25 bug: https://bugzilla.altlinux.org/show_bug.cgi?id=54323 gear: https://github.com/ProhorP/gettext-0.25 (Ответ для Usoltsev Petr на комментарий #14) > Переделал историю коммитов https://github.com/ProhorP/a2ps-4.15.6 > Финальный коммит сделал через "gear-commit -a" Всё в порядке, можно пробовать собирать на сборочнице. glebfm@, прошу перевести кандидата на стадию [3.6]. (Ответ для Usoltsev Petr на комментарий #15) > Сделал обновление gettext до 0.25 > bug: > https://bugzilla.altlinux.org/show_bug.cgi?id=54323 > gear: > https://github.com/ProhorP/gettext-0.25 К оформлению у меня претензий нет, но я плохо представляю возможные side-эффекты, от такого обновления. Лучше обсуждать обновление таких ключевых компонентов репозитория в devel@. К тому же в acl есть только @core, а это glebfm@ и ldv@. Сделал обновление пакета foomatic-db-engine с 4.0.12 на 4.0.13 gear: https://github.com/ProhorP/foomatic-db-engine-4.0.13 задача: https://bugzilla.altlinux.org/show_bug.cgi?id=54358 Собралось в sisyphus |
