* [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре @ 2021-08-16 1:37 Leonid Krivoshein 2021-08-16 5:22 ` Alexey V. Vissarionov ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Leonid Krivoshein @ 2021-08-16 1:37 UTC (permalink / raw) To: ALT Linux Team development discussions; +Cc: Aleksei Nikiforov Всем привет! Цель обсуждения: внести ясность и, при необходимости, корректировки на 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. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 1:37 [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре Leonid Krivoshein @ 2021-08-16 5:22 ` Alexey V. Vissarionov 2021-08-16 6:22 ` Andrey Cherepanov 2021-08-16 7:39 ` Anton Farygin 2 siblings, 0 replies; 10+ messages in thread From: Alexey V. Vissarionov @ 2021-08-16 5:22 UTC (permalink / raw) To: ALT Linux Team development discussions 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 1:37 [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре Leonid Krivoshein 2021-08-16 5:22 ` Alexey V. Vissarionov @ 2021-08-16 6:22 ` Andrey Cherepanov 2021-08-16 11:30 ` Leonid Krivoshein 2021-08-16 7:39 ` Anton Farygin 2 siblings, 1 reply; 10+ messages in thread From: Andrey Cherepanov @ 2021-08-16 6:22 UTC (permalink / raw) To: devel 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 6:22 ` Andrey Cherepanov @ 2021-08-16 11:30 ` Leonid Krivoshein 0 siblings, 0 replies; 10+ messages in thread From: Leonid Krivoshein @ 2021-08-16 11:30 UTC (permalink / raw) To: devel 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. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 1:37 [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре Leonid Krivoshein 2021-08-16 5:22 ` Alexey V. Vissarionov 2021-08-16 6:22 ` Andrey Cherepanov @ 2021-08-16 7:39 ` Anton Farygin 2021-08-16 11:42 ` Leonid Krivoshein 2 siblings, 1 reply; 10+ messages in thread From: Anton Farygin @ 2021-08-16 7:39 UTC (permalink / raw) To: devel Да, где встречаю и где осмысленно удаляю поле 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. И ещё очень много других статей на той же ВиКи... > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 7:39 ` Anton Farygin @ 2021-08-16 11:42 ` Leonid Krivoshein 2021-08-16 11:52 ` Anton Farygin 2021-08-16 13:06 ` Sergey V Turchin 0 siblings, 2 replies; 10+ messages in thread From: Leonid Krivoshein @ 2021-08-16 11:42 UTC (permalink / raw) To: devel К слову, эксперимент с локальной сборкой показал, что: - в спек, попадающий в .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. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 11:42 ` Leonid Krivoshein @ 2021-08-16 11:52 ` Anton Farygin 2021-08-16 12:16 ` Leonid Krivoshein 2021-08-16 13:06 ` Sergey V Turchin 1 sibling, 1 reply; 10+ messages in thread From: Anton Farygin @ 2021-08-16 11:52 UTC (permalink / raw) To: devel On 16.08.2021 14:42, Leonid Krivoshein wrote: > Взять, к примеру, бэкпортирование. Если я уберу или Packager или > поменяю его, автор обидится и правильно сделает. В спеке это поле > выглядит как авторство первоначального документа. Можно сравнить это > полями СУБД для табличных записей CREATOR и EDITOR. Получается, что > сейчас rpm -bb в каких-то случаях ведёт себя как CREATOR, в каких-то, > как EDITOR. CREATOR меняться не должен. Автор скорее обидится если ты бэкпортируешь пакет, в котором автор будет вписан в поле Packager. Если расценивать поле Packager как сопровождающий, то вот я лично не все свои пакеты готов сопровождать в части stable репозиториев. Поэтому, если кто-то взял и отправил собранный мною пакет из sisyphus в c9f1, то ответственность за сделанное должен нести именно тот, кто отправил, а никак не тот, кто упакетил в Sisyphus. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 11:52 ` Anton Farygin @ 2021-08-16 12:16 ` Leonid Krivoshein 2021-08-16 12:36 ` Anton Farygin 0 siblings, 1 reply; 10+ messages in thread From: Leonid Krivoshein @ 2021-08-16 12:16 UTC (permalink / raw) To: devel 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. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 12:16 ` Leonid Krivoshein @ 2021-08-16 12:36 ` Anton Farygin 0 siblings, 0 replies; 10+ messages in thread From: Anton Farygin @ 2021-08-16 12:36 UTC (permalink / raw) To: devel 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. > > Интересно, как это сейчас. > > Сейчас никак ;) ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре 2021-08-16 11:42 ` Leonid Krivoshein 2021-08-16 11:52 ` Anton Farygin @ 2021-08-16 13:06 ` Sergey V Turchin 1 sibling, 0 replies; 10+ messages in thread From: Sergey V Turchin @ 2021-08-16 13:06 UTC (permalink / raw) To: ALT Linux Team development discussions On Monday, 16 August 2021 14:42:26 MSK Leonid Krivoshein wrote: [...] > Взять, к примеру, бэкпортирование. Если я уберу или Packager или поменяю > его, автор обидится и правильно сделает. Я бы обиделся, если бы без моего ведома бэкпортировали пакет туда, куда нельзя(эту версию/сборку) по моему мнению, а если бэкпортировали туда, куда мне по барабану, то и что там именно, вообще не интересует. [...] -- Regards, Sergey. ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-08-16 13:06 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-16 1:37 [devel] Назначения поля Packager в спеке: как в Альте и как в Федоре Leonid Krivoshein 2021-08-16 5:22 ` Alexey V. Vissarionov 2021-08-16 6:22 ` Andrey Cherepanov 2021-08-16 11:30 ` Leonid Krivoshein 2021-08-16 7:39 ` Anton Farygin 2021-08-16 11:42 ` Leonid Krivoshein 2021-08-16 11:52 ` Anton Farygin 2021-08-16 12:16 ` Leonid Krivoshein 2021-08-16 12:36 ` Anton Farygin 2021-08-16 13:06 ` Sergey V Turchin
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git