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)
https://github.com/ProhorP/a2ps-4.15.6
(Ответ для 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
ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3.
Сделал исправление сборки пакета fonts-ttf-material-icons. gear: https://github.com/ProhorP/fonts-ttf-material-icons_4.0.0-alt1_17 Собралось в sisyphus
(Ответ для Gleb F-Malinovskiy на комментарий #19) > ssh ключ на gitery.alt зарегистрирован. > Адрес для пересылки создан. > > T/J/S -> 2.3. Кандидат готов отправлять пакеты на сборку в gyle. Прошу перевести его на [3.6].
ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6.
По поводу foomatic-db-engine: https://git.altlinux.org/tasks/388597/gears/100/git?p=git;a=blobdiff;f=foomatic-db-engine.spec;h=4985f66287c0a7409c371f9bff7b87f16c9978b1;hp=70bf3e4fe41a715222391a80956f912ff584d697;hb=4e65597610b7b9aff4e564ca91a89e6b4d010b57;hpb=f12e5084ba19a287117e501f9f4b0df8ed1bc5f2 Нужны пояснения по этому исправлению. Зачем была добавлена вот эта конструкция? > %ifarch i586 > %_prefix/lib/cups/driver/foomatic > %else > /lib/cups/driver/foomatic > %endif Должен работать такой вариант универсально: %_libexecdir/cups/driver/foomatic На самом деле это симлинк: $ ls -la /lib/cups/driver/foomatic lrwxrwxrwx 1 root root 25 дек 8 2015 /lib/cups/driver/foomatic -> /usr/bin/foomatic-ppdfile /usr/bin/foomatic-ppdfile - архитектурно-независимый скрипт на Perl. А /lib - это симлинк на /usr/lib
(Ответ для Usoltsev Petr на комментарий #20) > Сделал исправление сборки пакета fonts-ttf-material-icons. > gear: > https://github.com/ProhorP/fonts-ttf-material-icons_4.0.0-alt1_17 > Собралось в sisyphus Approved.
a2ps - Approved.
(Ответ для Andrey Limachko на комментарий #23) > По поводу foomatic-db-engine: > https://git.altlinux.org/tasks/388597/gears/100/git?p=git;a=blobdiff; > f=foomatic-db-engine.spec;h=4985f66287c0a7409c371f9bff7b87f16c9978b1; > hp=70bf3e4fe41a715222391a80956f912ff584d697; > hb=4e65597610b7b9aff4e564ca91a89e6b4d010b57; > hpb=f12e5084ba19a287117e501f9f4b0df8ed1bc5f2 > Нужны пояснения по этому исправлению. > Зачем была добавлена вот эта конструкция? > > %ifarch i586 > > %_prefix/lib/cups/driver/foomatic > > %else > > /lib/cups/driver/foomatic > > %endif > Должен работать такой вариант универсально: > %_libexecdir/cups/driver/foomatic Запустил новую задачу в gyle #389162, она завершилась с ошибкой: [aarch64] error: No such file or directory: /usr/src/tmp/foomatic-db-engine-buildroot/usr/lib/cups/driver/foomatic В одной архитектуре путь: /usr/src/tmp/foomatic-db-engine-buildroot/usr/lib/cups/driver/foomatic в другой: /usr/src/tmp/foomatic-db-engine-buildroot/lib/cups/driver/foomatic поэтому использовал "%ifarch i586" > > На самом деле это симлинк: > $ ls -la /lib/cups/driver/foomatic > lrwxrwxrwx 1 root root 25 дек 8 2015 /lib/cups/driver/foomatic -> > /usr/bin/foomatic-ppdfile > > /usr/bin/foomatic-ppdfile - архитектурно-независимый скрипт на Perl. > А /lib - это симлинк на /usr/lib
Вместо такой конструкции: > %ifarch i586 > %_prefix/lib/cups/driver/foomatic > %else > /lib/cups/driver/foomatic > %endif лучше использовать такое: diff --git a/foomatic-db-engine.spec b/foomatic-db-engine.spec index d6074b6..4d346ee 100644 --- a/foomatic-db-engine.spec +++ b/foomatic-db-engine.spec @@ -42,6 +42,7 @@ for printer administration, and for printing. %build export SH=/bin/bash +export LIB_CUPS=%_libexecdir/cups %configure %make
Спасибо это сработало. Задача сборки #391005. Получается что export LIB_CUPS=%_libexecdir/cups задает путь для макроса FM_PATH_DIRS(LIB_CUPS,cups,/usr/lib/cups,$LIBSEARCHPATH) из файла configure.ac и туда помещается "архитектурно-независимый скрипт на Perl /usr/bin/foomatic-ppdfile" при сборке И теперь устанавливаемый файл %_libexecdir/cups/driver/foomatic будет найден для любой архитектуры
(Ответ для Usoltsev Petr на комментарий #28) > Спасибо это сработало. > Задача сборки #391005. > Получается что > export LIB_CUPS=%_libexecdir/cups > задает путь для макроса > FM_PATH_DIRS(LIB_CUPS,cups,/usr/lib/cups,$LIBSEARCHPATH) из файла > configure.ac и туда помещается "архитектурно-независимый скрипт на Perl > /usr/bin/foomatic-ppdfile" при сборке > И теперь устанавливаемый файл %_libexecdir/cups/driver/foomatic будет найден > для любой архитектуры Уехало в sisyphus.
(Ответ для Andrey Limachko на комментарий #17) > (Ответ для 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@. На этапе Join [3.6] должна была появиться возможность писать в рассылку devel@. Самое время написать туда про обновлённый gettext 0.25.
Рекомендую кандидату познакомиться с другими схемами сборки пакетов. К примеру: Сборка из тэга с alt-специфичными файлами и содержимым тэга в основной ветке: https://packages.altlinux.org/ru/sisyphus/srpms/shfmt/ Сборка из тэга с alt-специфичными файлами в основной ветке. Тэг смержен в основную ветку с git merge -s ours, хэш коммита тэга прописан в .gear/tags/list: https://packages.altlinux.org/ru/sisyphus/srpms/kea/
(Ответ для Andrey Limachko на комментарий #30) > На этапе Join [3.6] должна была появиться возможность писать в рассылку > devel@. Самое время написать туда про обновлённый gettext 0.25. Спасибо, написал
(Ответ для Andrey Limachko на комментарий #31) > Рекомендую кандидату познакомиться с другими схемами сборки пакетов. > К примеру: > > Сборка из тэга с alt-специфичными файлами и содержимым тэга в основной ветке: > https://packages.altlinux.org/ru/sisyphus/srpms/shfmt/ > > Сборка из тэга с alt-специфичными файлами в основной ветке. Тэг смержен в > основную ветку с git merge -s ours, хэш коммита тэга прописан в > .gear/tags/list: > https://packages.altlinux.org/ru/sisyphus/srpms/kea/ Хорошо, продолжаю join
