Bug 52347

Summary: join usoltsevpv@
Product: Team Accounts Reporter: Usoltsev Petr <usoltsevpv>
Component: joinAssignee: 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 Flags
ssh ключ
none
gpg ключ
none
gpg ключ @altlinux none

Description Usoltsev Petr 2024-12-05 07:33:15 MSK

    
Comment 1 Usoltsev Petr 2024-12-05 07:38:35 MSK
Created attachment 17326 [details]
ssh ключ
Comment 2 Usoltsev Petr 2024-12-05 07:38:59 MSK
Created attachment 17327 [details]
gpg ключ
Comment 3 Usoltsev Petr 2024-12-05 07:40:31 MSK
ментор - liannnix@
Comment 4 Usoltsev Petr 2024-12-07 08:20:53 MSK
Created attachment 17349 [details]
gpg ключ @altlinux
Comment 5 Usoltsev Petr 2024-12-07 08:22:25 MSK
Псевдоним: usoltsevpv
Адрес пересылки: usoltsev05@mail.ru
Ментор: Андрей Лимачко <liannnix@altlinux.org>
Цель: Научиться собирать пакеты
Comment 6 Gleb F-Malinovskiy 2024-12-11 16:39:34 MSK
Ключи в порядке, для продолжения нужен ментор.
Comment 7 Usoltsev Petr 2025-03-10 10:45:37 MSK
Завел задачу по исправлению сборки a2ps:
https://bugzilla.altlinux.org/53318
Исправление опубликовал тут:
https://github.com/ProhorP/a2ps
Comment 8 Andrey Limachko 2025-03-10 21:40:23 MSK
(Ответ для 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

Может, ещё что пропустил. Когда всё в кашу в одном коммите, то разбираться сложно.