[devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Leonid Krivoshein]

Всем привет!


Цель обсуждения: внести ясность и, при необходимости, корректировки на 
ALT ВиКи.

В Fedora Packagin GuideLines [1] сказано:
The Copyright:, Packager:, Vendor: and PreReq: tags MUST NOT be used.

Такую же рекомендацию уже второй раз встречаю за последнее время: не 
надо использовать поле Packager *в спеке*, т.к. наша сборочница его и 
так автоматически проставляет. Однако git grep по spec.git by vt@ 
покажет огромное число пакетов, где поле Packager прописано. Мне 
кажется, что назначение поля Packager *в спеке* -- обозначить, кто 
*первый* опакетил данное ПО в Сизиф или какая группа разработки за него 
сейчас отвечает. Конечно, можно это выяснить и более длинным путём, но 
так проще и интуитивно соответствует программистской логике. Исходя из 
этого в наших полиси должны быть прописаны все случаи, когда изменение 
этого поля уместно, и когда его можно действительно не указывать. Сейчас 
это именно так, судя по ВиКи. Если поле и правда лишнее, в чём лично я 
сильно сомневаюсь, стоит хотя бы предупреждение выдавать.


Ссылки:
1. 
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections
2. https://www.altlinux.org/ALT_Packaging_HOWTO#Порядок_тэгов
3. https://www.altlinux.org/ACL#ACL
4. https://www.altlinux.org/NMU
5. https://www.altlinux.org/Сборка_модулей_ядра
6. https://www.altlinux.org/Backports_Policy#Исправления_spec-файла
7. https://www.altlinux.org/Packager_Teams
8. https://www.altlinux.org/SampleSpecs/cmakeprogram -- и таких примеров 
там десятки.
9. И ещё очень много других статей на той же ВиКи...


-- 
Best regards,
Leonid Krivoshein.

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Alexey V. Vissarionov]

On 2021-08-16 04:37:02 +0300, Leonid Krivoshein wrote:

 > Цель обсуждения: внести ясность и, при необходимости,
 > корректировки на ALT ВиКи.
 > В Fedora Packagin GuideLines [1] сказано: The Copyright:,
 > Packager:, Vendor: and PreReq: tags MUST NOT be used.

И в общем-то правильно сказано.

 > Такую же рекомендацию

Это не рекомендация - это безусловное требование. Подробности
в RFC-2119.

 > уже второй раз встречаю за последнее время: не надо
 > использовать поле Packager *в спеке*, т.к. наша сборочница
 > его и так автоматически проставляет.

Если оно нам для чего-то нужно - пусть проставляет. Но в спеке
ему совершенно точно не место.

 > Однако git grep по spec.git by vt@ покажет огромное число
 > пакетов, где поле Packager прописано.

Это нужно исправить. Пусть даже и не срочно.

 > Мне кажется, что назначение поля Packager *в спеке*
 > -- обозначить, кто *первый* опакетил данное ПО в Сизиф или
 > какая группа разработки за него сейчас отвечает.

Для этого можно (и более правильно) использовать changelog.

 > Конечно, можно это выяснить и более длинным путём, но так
 > проще и интуитивно соответствует программистской логике.

Если уже смотрим в спек - можно и до ченджлога долистать.

 > Исходя из этого в наших полиси должны быть прописаны все
 > случаи, когда изменение этого поля уместно, и когда его
 > можно действительно не указывать. Сейчас это именно так,
 > судя по ВиКи. Если поле и правда лишнее, в чём лично я
 > сильно сомневаюсь, стоит хотя бы предупреждение выдавать.

Поле действительно лишнее. Ломать сборку из-за него, конечно,
не нужно, но если в спек были внесены изменения (пусть даже
release bump) - требовать убрать deprecated поля будет вполне
уместно.


-- 
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Andrey Cherepanov]

16.08.2021 04:37, Leonid Krivoshein пишет:
> Всем привет!
>
>
> Цель обсуждения: внести ясность и, при необходимости, корректировки на 
> ALT ВиКи.
>
> В Fedora Packagin GuideLines [1] сказано:
> The Copyright:, Packager:, Vendor: and PreReq: tags MUST NOT be used.
>
> Такую же рекомендацию уже второй раз встречаю за последнее время: не 
> надо использовать поле Packager *в спеке*, т.к. наша сборочница его и 
> так автоматически проставляет. Однако git grep по spec.git by vt@ 
> покажет огромное число пакетов, где поле Packager прописано. Мне 
> кажется, что назначение поля Packager *в спеке* -- обозначить, кто 
> *первый* опакетил данное ПО в Сизиф или какая группа разработки за 
> него сейчас отвечает. Конечно, можно это выяснить и более длинным 
> путём, но так проще и интуитивно соответствует программистской логике. 
> Исходя из этого в наших полиси должны быть прописаны все случаи, когда 
> изменение этого поля уместно, и когда его можно действительно не 
> указывать. Сейчас это именно так, судя по ВиКи. Если поле и правда 
> лишнее, в чём лично я сильно сомневаюсь, стоит хотя бы предупреждение 
> выдавать.
Я придерживаюсь практики, что Packager - тот, кто несёт ответственность 
за пакет. По разным причинам лидера в ACL далеко не всегда можно 
учитывать и не так просто сменить. У меня вопрос: у нас всё прекрасно и 
заняться нечем, что стоит это обсуждать?

-- 
Andrey Cherepanov
cas@altlinux.org

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Leonid Krivoshein]

16.08.2021 9:22, Andrey Cherepanov пишет:
> 16.08.2021 04:37, Leonid Krivoshein пишет:
>> Всем привет!
>>
>>
>> Цель обсуждения: внести ясность и, при необходимости, корректировки 
>> на ALT ВиКи.
>>
>> В Fedora Packagin GuideLines [1] сказано:
>> The Copyright:, Packager:, Vendor: and PreReq: tags MUST NOT be used.
>>
>> Такую же рекомендацию уже второй раз встречаю за последнее время: не 
>> надо использовать поле Packager *в спеке*, т.к. наша сборочница его и 
>> так автоматически проставляет. Однако git grep по spec.git by vt@ 
>> покажет огромное число пакетов, где поле Packager прописано. Мне 
>> кажется, что назначение поля Packager *в спеке* -- обозначить, кто 
>> *первый* опакетил данное ПО в Сизиф или какая группа разработки за 
>> него сейчас отвечает. Конечно, можно это выяснить и более длинным 
>> путём, но так проще и интуитивно соответствует программистской 
>> логике. Исходя из этого в наших полиси должны быть прописаны все 
>> случаи, когда изменение этого поля уместно, и когда его можно 
>> действительно не указывать. Сейчас это именно так, судя по ВиКи. Если 
>> поле и правда лишнее, в чём лично я сильно сомневаюсь, стоит хотя бы 
>> предупреждение выдавать.
> Я придерживаюсь практики, что Packager - тот, кто несёт 
> ответственность за пакет. По разным причинам лидера в ACL далеко не 
> всегда можно учитывать и не так просто сменить. У меня вопрос: у нас 
> всё прекрасно и заняться нечем, что стоит это обсуждать?
>

Раз у каждого своя практика, а на ВиКи неверная информация на десятках 
страниц, обсудить, чтобы хотя бы давать ссылку на результат обсуждения, 
было бы правильно.


-- 
Best regards,
Leonid Krivoshein.

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Anton Farygin]

Да, где встречаю и где осмысленно удаляю поле Packager из спека.

В целом придерживаюсь рекомендации, что поле Packager скорее вредное, 
чем полезное.
А настоящего ментейнера, отвечающего за пакет можно определять по changelog.


На wiki надо бы тоже поправить.


On 16.08.2021 04:37, Leonid Krivoshein wrote:
>
> Цель обсуждения: внести ясность и, при необходимости, корректировки на 
> ALT ВиКи.
>
> В Fedora Packagin GuideLines [1] сказано:
> The Copyright:, Packager:, Vendor: and PreReq: tags MUST NOT be used.
>
> Такую же рекомендацию уже второй раз встречаю за последнее время: не 
> надо использовать поле Packager *в спеке*, т.к. наша сборочница его и 
> так автоматически проставляет. Однако git grep по spec.git by vt@ 
> покажет огромное число пакетов, где поле Packager прописано. Мне 
> кажется, что назначение поля Packager *в спеке* -- обозначить, кто 
> *первый* опакетил данное ПО в Сизиф или какая группа разработки за 
> него сейчас отвечает. Конечно, можно это выяснить и более длинным 
> путём, но так проще и интуитивно соответствует программистской логике. 
> Исходя из этого в наших полиси должны быть прописаны все случаи, когда 
> изменение этого поля уместно, и когда его можно действительно не 
> указывать. Сейчас это именно так, судя по ВиКи. Если поле и правда 
> лишнее, в чём лично я сильно сомневаюсь, стоит хотя бы предупреждение 
> выдавать.
>
>
> Ссылки:
> 1. 
> https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections
> 2. https://www.altlinux.org/ALT_Packaging_HOWTO#Порядок_тэгов
> 3. https://www.altlinux.org/ACL#ACL
> 4. https://www.altlinux.org/NMU
> 5. https://www.altlinux.org/Сборка_модулей_ядра
> 6. https://www.altlinux.org/Backports_Policy#Исправления_spec-файла
> 7. https://www.altlinux.org/Packager_Teams
> 8. https://www.altlinux.org/SampleSpecs/cmakeprogram -- и таких 
> примеров там десятки.
> 9. И ещё очень много других статей на той же ВиКи...
>
>

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Leonid Krivoshein]

К слову, эксперимент с локальной сборкой показал, что:

- в спек, попадающий в .src.rpm, поле Packager, если есть, значение 
заменяется на текущего сборщика пакета;
- в поле Packager в .rpm и .src.rpm прописывается значение поля Packager 
из исходного гита, а не текущего сборщика пакета;
- только если не указывать поле Packager в исходном гите, только тогда 
пакеты собираются с текущим сборщикам.
- мне удалось собрать пакет с Packager: John Doe <agent007@altlinux.org>;

Первая особенность интересна, поскольку незаметно для сборщика и автора 
("тихо") заменяется "исходник".

Исходя из сказанного:

- наши полиси на ВиКи противоречат тому, что в Федоре;
- мне эти полиси кажутся логичными, но не везде достаточными;

Взять, к примеру, бэкпортирование. Если я уберу или Packager или поменяю 
его, автор обидится и правильно сделает. В спеке это поле выглядит как 
авторство первоначального документа. Можно сравнить это полями СУБД для 
табличных записей CREATOR и EDITOR. Получается, что сейчас rpm -bb в 
каких-то случаях ведёт себя как CREATOR, в каких-то, как EDITOR. CREATOR 
меняться не должен.


16.08.2021 10:39, Anton Farygin пишет:
> Да, где встречаю и где осмысленно удаляю поле Packager из спека.
>
> В целом придерживаюсь рекомендации, что поле Packager скорее вредное, 
> чем полезное.
> А настоящего ментейнера, отвечающего за пакет можно определять по 
> changelog.
>
>
> На wiki надо бы тоже поправить.
>
>
> On 16.08.2021 04:37, Leonid Krivoshein wrote:
>>
>> Цель обсуждения: внести ясность и, при необходимости, корректировки 
>> на ALT ВиКи.
>>
>> В Fedora Packagin GuideLines [1] сказано:
>> The Copyright:, Packager:, Vendor: and PreReq: tags MUST NOT be used.
>>
>> Такую же рекомендацию уже второй раз встречаю за последнее время: не 
>> надо использовать поле Packager *в спеке*, т.к. наша сборочница его и 
>> так автоматически проставляет. Однако git grep по spec.git by vt@ 
>> покажет огромное число пакетов, где поле Packager прописано. Мне 
>> кажется, что назначение поля Packager *в спеке* -- обозначить, кто 
>> *первый* опакетил данное ПО в Сизиф или какая группа разработки за 
>> него сейчас отвечает. Конечно, можно это выяснить и более длинным 
>> путём, но так проще и интуитивно соответствует программистской 
>> логике. Исходя из этого в наших полиси должны быть прописаны все 
>> случаи, когда изменение этого поля уместно, и когда его можно 
>> действительно не указывать. Сейчас это именно так, судя по ВиКи. Если 
>> поле и правда лишнее, в чём лично я сильно сомневаюсь, стоит хотя бы 
>> предупреждение выдавать.
>>
>>
>> Ссылки:
>> 1. 
>> https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections
>> 2. https://www.altlinux.org/ALT_Packaging_HOWTO#Порядок_тэгов
>> 3. https://www.altlinux.org/ACL#ACL
>> 4. https://www.altlinux.org/NMU
>> 5. https://www.altlinux.org/Сборка_модулей_ядра
>> 6. https://www.altlinux.org/Backports_Policy#Исправления_spec-файла
>> 7. https://www.altlinux.org/Packager_Teams
>> 8. https://www.altlinux.org/SampleSpecs/cmakeprogram -- и таких 
>> примеров там десятки.
>> 9. И ещё очень много других статей на той же ВиКи...
>>
>>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

-- 
Best regards,
Leonid Krivoshein.

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Anton Farygin]

On 16.08.2021 14:42, Leonid Krivoshein wrote:
> Взять, к примеру, бэкпортирование. Если я уберу или Packager или 
> поменяю его, автор обидится и правильно сделает. В спеке это поле 
> выглядит как авторство первоначального документа. Можно сравнить это 
> полями СУБД для табличных записей CREATOR и EDITOR. Получается, что 
> сейчас rpm -bb в каких-то случаях ведёт себя как CREATOR, в каких-то, 
> как EDITOR. CREATOR меняться не должен. 

Автор скорее обидится если ты бэкпортируешь пакет, в котором автор будет 
вписан в поле Packager.

Если расценивать поле Packager как сопровождающий, то вот я лично не все 
свои пакеты готов сопровождать в части stable репозиториев.

Поэтому, если кто-то взял и отправил собранный мною пакет из sisyphus в 
c9f1, то ответственность за сделанное должен нести именно тот, кто 
отправил, а никак не тот, кто упакетил в Sisyphus.

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Leonid Krivoshein]

16.08.2021 14:52, Anton Farygin пишет:
> On 16.08.2021 14:42, Leonid Krivoshein wrote:
>> Взять, к примеру, бэкпортирование. Если я уберу или Packager или 
>> поменяю его, автор обидится и правильно сделает. В спеке это поле 
>> выглядит как авторство первоначального документа. Можно сравнить это 
>> полями СУБД для табличных записей CREATOR и EDITOR. Получается, что 
>> сейчас rpm -bb в каких-то случаях ведёт себя как CREATOR, в каких-то, 
>> как EDITOR. CREATOR меняться не должен. 
>
> Автор скорее обидится если ты бэкпортируешь пакет, в котором автор 
> будет вписан в поле Packager.
>

А как иначе? Нарушать полиси?
https://www.altlinux.org/Backports_Policy#Исправления_spec-файла

У нас везде как раз написано, что поле должно быть и менять нельзя, 
только в определённых случаях. По факту, как вижу, практика самая 
разная. Надо либо все эти полиси менять, чтобы не вводить людей в 
заблуждение, либо работать строго по ним.


> Если расценивать поле Packager как сопровождающий, то вот я лично не 
> все свои пакеты готов сопровождать в части stable репозиториев.
>

Как "последний собравший пакет". И отдельный вопрос с копированием 
тестовой пересборкой, где вроде всё ОК и даже последний коммит от тебя, 
но кто отвечает за полученный результат в другом бранче...


> Поэтому, если кто-то взял и отправил собранный мною пакет из sisyphus 
> в c9f1, то ответственность за сделанное должен нести именно тот, кто 
> отправил, а никак не тот, кто упакетил в Sisyphus.

Интересно, как это сейчас.


-- 
Best regards,
Leonid Krivoshein.

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Anton Farygin]

On 16.08.2021 15:16, Leonid Krivoshein wrote:
>
> 16.08.2021 14:52, Anton Farygin пишет:
>> On 16.08.2021 14:42, Leonid Krivoshein wrote:
>>> Взять, к примеру, бэкпортирование. Если я уберу или Packager или 
>>> поменяю его, автор обидится и правильно сделает. В спеке это поле 
>>> выглядит как авторство первоначального документа. Можно сравнить это 
>>> полями СУБД для табличных записей CREATOR и EDITOR. Получается, что 
>>> сейчас rpm -bb в каких-то случаях ведёт себя как CREATOR, в 
>>> каких-то, как EDITOR. CREATOR меняться не должен. 
>>
>> Автор скорее обидится если ты бэкпортируешь пакет, в котором автор 
>> будет вписан в поле Packager.
>>
>
> А как иначе? Нарушать полиси?
> https://www.altlinux.org/Backports_Policy#Исправления_spec-файла
>
> У нас везде как раз написано, что поле должно быть и менять нельзя, 
> только в определённых случаях. По факту, как вижу, практика самая 
> разная. Надо либо все эти полиси менять, чтобы не вводить людей в 
> заблуждение, либо работать строго по ним.
Policy многие устаревают. Вообще, работа по поддержанию Policy в 
актуальном состоянии не ведётся, а это довольно серьёзное занятие, 
требующее заметных ресурсов.
>
>
>> Если расценивать поле Packager как сопровождающий, то вот я лично не 
>> все свои пакеты готов сопровождать в части stable репозиториев.
>>
>
> Как "последний собравший пакет". И отдельный вопрос с копированием 
> тестовой пересборкой, где вроде всё ОК и даже последний коммит от 
> тебя, но кто отвечает за полученный результат в другом бранче...
>
>
>> Поэтому, если кто-то взял и отправил собранный мною пакет из sisyphus 
>> в c9f1, то ответственность за сделанное должен нести именно тот, кто 
>> отправил, а никак не тот, кто упакетил в Sisyphus.
>
> Интересно, как это сейчас.
>
>
Сейчас никак ;)

Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре

[Sergey V Turchin]

On Monday, 16 August 2021 14:42:26 MSK Leonid Krivoshein wrote:

[...]
> Взять, к примеру, бэкпортирование. Если я уберу или Packager или поменяю
> его, автор обидится и правильно сделает. 
Я бы обиделся, если бы без моего ведома бэкпортировали пакет туда, куда 
нельзя(эту версию/сборку) по моему мнению, а если бэкпортировали туда, куда 
мне по барабану, то и что там именно, вообще не интересует.

[...]

-- 
Regards, Sergey.