[devel] Вмешательство в пакеты других мантейнеров

[devel] Вмешательство в пакеты других мантейнеров

[Vitaly Lipatov]

Несколько раз замечал, что в пакетах по неизвестным причинам меняют 
схему сборки.

Я вижу несколько причин, по которым не стоит так делать в пакетах, 
которые ведёт другой мантейнер:
1. Не всегда урезанный репозиторий, выложенный в /gears после сборки, 
позволяет вести сопровождение пакета с прежним удобством (никто не 
воссоздаст все эти веточки, которые мержат или создают из них патчи).

2. Иногда обновление становится просто сложным и запутанным, см.
https://lists.altlinux.org/pipermail/devel/2023-July/217702.html

3. Вообще неправильно вторгаться в то, как устроены пакеты. При этом, 
конечно, помощь в сборке новых версий это прекрасно.

4. Ничего сложного нет в том, чтобы запустить
$ rpmrb <новая версия>
и тем самым обновить пакет.


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

%changelog
* Wed Jul 19 2023 Stanislav Levin <slev@altlinux.org> 2.0.8-alt1
- 2.0.4 -> 2.0.8.

* Thu May 11 2023 Vitaly Lipatov <lav@altlinux.ru> 2.0.4-alt1
- new version 2.0.4 (with rpmrb script)
- update requires


В данном случае за лаконичным описанием 2.0.4 -> 2.0.8 кроется большая 
работа, проделанная Станиславом, и, очевидно, теперь он мантейнер этого 
пакета, хотя по ACL это никак не скажешь.
Мне кажется, произведённые изменения заслуживают более обширного 
описание в changelog, ведь там столько интересного:

     spec: Enable testing

     spec: Apply a cumulative patch

     spec: Package conan-server

     conan-server is built from the same repo and ships exactly the
     same Python modules/packages as conan (client) + conans.server
     package.

     spec: Enable auto provides back
     It's unclear why it was disabled previously.

     spec: Map PyPI name to distro's one
     See 
https://www.altlinux.org/Management_of_Python_dependencies_sources#Mapping_project_names_to_distro_names
     for details.

     spec: Drop no longer needed sedding of node-semver
     Backported from 793dfaecf07381e993b84d3b82c157ccf50a6399 (2.0.0).

     spec: Deduplicate packaging of common license
     See https://www.altlinux.org/Spec#License for details.

     packaging: Make use of external source of upstream's dependencies
     See 
https://www.altlinux.org/Management_of_Python_dependencies_sources
     for details.

     spec: Switch to PEP517 build
     Build Python project with modern PEP517 interface instead of legacy
     setuptools' one.

     See 
https://www.altlinux.org/Python_packaging_guide#Migration_to_PEP517_in_RPM_specfile
     for details.

     spec: Don't allow implicit unpackaged files
     In general, it's a common error to have implicit unpackaged files.

     spec: Fill in RPM VCS tag
     It can be used by external tools.

     spec: Drop obsolete RPM Packager tag
     spec: Reorder RPM tags according to recommended order

     See 
https://www.altlinux.org/%D0%9E%D0%B1%D1%89%D0%B8%D0%B5_%D0%BF%D1%80%D0%B0%D0%B2%D0%B8%D0%BB%D0%B0_%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D0%BD%D0%B8%D1%8F_%D1%81%D0%BF%D0%B5%D0%BA_%D1%84%D0%B0%D0%B9%D0%BB%D0%BE%D0%B2_%D0%B2>
     for details.

     Merge tag '2.0.5' into sisyphus




Заодно могу ответить на
commit 8c385d9ee9390630e10ede70f231e1275afc6825
Author: Stanislav Levin <slev@altlinux.org>
Date:   Wed May 24 13:13:40 2023 +0300

     spec: Enable auto provides back

     It's unclear why it was disabled previously.

Ничего хорошего нет в том, что пакет с программой провайдит какие-то 
модули или библиотеки. Поэтому я считаю хорошим правилом делать 
AutoProv: no для пакетов с программами.

-- 
С уважением,
Виталий Липатов,
ALT Linux Team

Re: [devel] Вмешательство в пакеты других мантейнеров

[Stanislav Levin]

[-- Attachment #1.1: Type: text/plain, Size: 2152 bytes --]



05.08.2023 22:22, Vitaly Lipatov пишет:
> Несколько раз замечал, что в пакетах по неизвестным причинам меняют 
> схему сборки.
> 
> Я вижу несколько причин, по которым не стоит так делать в пакетах, 
> которые ведёт другой мантейнер:
> 1. Не всегда урезанный репозиторий, выложенный в /gears после сборки, 
> позволяет вести сопровождение пакета с прежним удобством (никто не 
> воссоздаст все эти веточки, которые мержат или создают из них патчи).
> 
> 2. Иногда обновление становится просто сложным и запутанным, см.
> https://lists.altlinux.org/pipermail/devel/2023-July/217702.html
> 
> 3. Вообще неправильно вторгаться в то, как устроены пакеты. При этом, 
> конечно, помощь в сборке новых версий это прекрасно.
> 
> 4. Ничего сложного нет в том, чтобы запустить
> $ rpmrb <новая версия>
> и тем самым обновить пакет.
> 

1) наличие @everybody в acl = приглашение к участию в жизни пакета 
любого желающего.
2) схема пакетирования - личное предпочтение сопровождающего. Например, 
я меняю схему пакетирования на удобную мне, если нет никаких 
противопоказаний.
3) мне не интересно разбираться с прочими инструментами для управления 
пакетом, если в этом нет необходимости.
4) мне не хочется обсуждать субъективное содержимое changelog в этой 
рассылке


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]

Re: [devel] Вмешательство в пакеты других мантейнеров

[Alexey V. Vissarionov]

Good ${greeting_time}!

On 2023-08-07 11:58:49 +0300, Stanislav Levin wrote:

 > 1) наличие @everybody в acl = приглашение к участию в жизни
 > пакета любого желающего.

Не приглашение, а разрешение. Прыгать через голову живого и
доступного действующего мейнтейнера - фу, не надо так делать.

 > 2) схема пакетирования - личное предпочтение сопровождающего.
 > Например, я меняю схему пакетирования на удобную мне, если нет
 > никаких противопоказаний.

Отсутствие явного одобрения от живого действующего мейнтейнера -
это даже не противопоказание, а блокер.

 > 3) мне не интересно разбираться с прочими инструментами для
 > управления пакетом, если в этом нет необходимости.

А лезть ногами в "чужие" пакеты интересно?

Не в заброшенные (когда либо с действующим мейнтейнером нет связи,
либо мейнтейнера вообще нет, а в ACL указан nobody), а в чужие?

Я в "своих" пакетах пришел к варианту, когда вмешательство на
всякий случай разрешено группе QA - например, в ситуации, когда
я уехал отдыхать куда-то в совсем дикие места, и именно в этот
момент была найдена и опубликована zero-day уязвимость. При этом
я исхожу из того, что у участников этой группы достаточно ума,
чтобы не лезть куда-то без особой нужды. За все время была всего
одна ситуация, когда один участник QA все же полез в мой пакет
ногами - ну что ж, вандализм откатил, человека вразумил.

 > 4) мне не хочется обсуждать субъективное содержимое changelog
 > в этой рассылке

Может, лучше не давать для этого ни причин, ни поводов?


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

Re: [devel] Вмешательство в пакеты других мантейнеров

[Vitaly Lipatov]

Stanislav Levin писал(а) 7.8.23 11:58:
...
> 1) наличие @everybody в acl = приглашение к участию в жизни пакета 
> любого желающего.
> 2) схема пакетирования - личное предпочтение сопровождающего. Например, 
> я меняю схему пакетирования на удобную мне, если нет никаких 
> противопоказаний.
Вы каким-то образом резко перешли от участия в жизни пакета к тому, что 
стали его сопровождающим.
@everybody на пакетах у нас стоит по умолчанию, но это же не значит, что 
можно мешать мантейнеру и делать что угодно. Ведь можно и обсудить свои 
планы на пакет. А если вы хотите стать его сопровождающим, потому что он 
вам нужнее и вы в нём лучше разбираетесь — тоже об этом написать можно.

> 3) мне не интересно разбираться с прочими инструментами для управления 
> пакетом, если в этом нет необходимости.
Но при этом вы считаете, что мне интересно разбираться с тем, какая 
схема пакетирования удобна вам?
Дело в том, что такое обращение с пакетом похоже на рейдерский захват: 
фактически, исправив пакет максимально «под себя», вы его забрали.

> 4) мне не хочется обсуждать субъективное содержимое changelog в этой 
> рассылке
В смысле? Эта рассылка как раз для того, чтобы обсуждать разработку 
репозитория участниками Тим.

Вот вы написали какие-то интересные статьи на непонятном языке, в 
соответствии с ними исправляете пакеты
https://www.altlinux.org/Management_of_Python_dependencies_sources
https://www.altlinux.org/Python_packaging_guide#Migration_to_PEP517_in_RPM_specfile

конечно, интересно довести всё это до обновления Python Policy, чтобы мы 
все работали над пакетами python в едином стиле.

Подводя итог: своим письмом я хотел вам сообщить, что вы перешли 
некоторые границы, на мой взгляд. Но это не так страшно, всегда можно 
договориться, я надеюсь, и мы не в диком поле, где нужно обязательно 
следить за воротами и @everybody, иначе лошадей угонят и дом подожгут.


-- 
С уважением,
Виталий Липатов,
ALT Linux Team

Re: [devel] Вмешательство в пакеты других мантейнеров

[Sergey Afonin]

On Monday 07 August 2023, Stanislav Levin wrote:

> 1) наличие @everybody в acl = приглашение к участию в жизни пакета 
> любого желающего.

@everybody не означает @nobody.

> 2) схема пакетирования - личное предпочтение сопровождающего.

Да. Но не добровольных помощников. Если меняется схема, следует спросить.
Может быть даже никто и не будет против, но спросить следует.

-- 
С уважением, Сергей Афонин.

[devel] Re: Вмешательство в пакеты других мантейнеров

[Sergey V Turchin]

On Monday, 14 August 2023 12:26:47 MSK Sergey Afonin wrote:
> On Monday 07 August 2023, Stanislav Levin wrote:
> > 1) наличие @everybody в acl = приглашение к участию в жизни пакета
> > любого желающего.
> @everybody не означает @nobody.
Думаю, пока нет чёткого определения, каждый волен трактовать по своему.

[...]

-- 
Regards, Sergey.

Re: [devel] Вмешательство в пакеты других мантейнеров

[Vitaly Lipatov]

Sergey V Turchin писал(а) 14.8.23 13:32:
> On Monday, 14 August 2023 12:26:47 MSK Sergey Afonin wrote:
>> On Monday 07 August 2023, Stanislav Levin wrote:
>> > 1) наличие @everybody в acl = приглашение к участию в жизни пакета
>> > любого желающего.
>> @everybody не означает @nobody.
> Думаю, пока нет чёткого определения, каждый волен трактовать по своему.
Так мы здесь и формулируем общепринятую трактовку. После того, как о ней 
стало известно, уже не стоит трактовать по-своему.

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

Для меня @nobody это бесхозный пакет. Это вроде очевидно.

Наличие @everybody я трактую от «приветствуются NMU и обновление пакета» 
(если видно, что мантейнер давно пакетом не занимался) до «можно 
отправить NMU, но стоит спросить владельца пакета, всё ли его 
устраивает».

Отсутствие @everybody это сигнал в диапазоне от того, что 
сопровождающему удобнее контролировать сборку до того, что мало кому 
можно доверить вносить изменения в такой ценный пакет.

-- 
С уважением,
Виталий Липатов,
ALT Linux Team

[devel] Re: Вмешательство в пакеты других мантейнеров

[Sergey V Turchin]

On Monday, 14 August 2023 21:51:00 MSK Vitaly Lipatov wrote:
> Sergey V Turchin писал(а) 14.8.23 13:32:
> > On Monday, 14 August 2023 12:26:47 MSK Sergey Afonin wrote:
> >> On Monday 07 August 2023, Stanislav Levin wrote:
> >> > 1) наличие @everybody в acl = приглашение к участию в жизни пакета
> >> > любого желающего.
> >> 
> >> @everybody не означает @nobody.
> > 
> > Думаю, пока нет чёткого определения, каждый волен трактовать по своему.
> 
> Так мы здесь и формулируем общепринятую трактовку.
"Отучаемся говорить за всех" * 2
Со своих пакетов я давным-давно убрал @everybody, когда мне надоели правки без 
спроса, чего и всем советую. И не надо разводить болтологию.

[...]


-- 
Regards, Sergey.

Re: [devel] Вмешательство в пакеты других мантейнеров

[Vitaly Lipatov]

Sergey V Turchin писал(а) 15.8.23 11:35:
...
>> Так мы здесь и формулируем общепринятую трактовку.
> "Отучаемся говорить за всех" * 2
Если вы покинули Team, это не значит, что теперь ваше отдельное мнение 
что-то значит.

-- 
С уважением,
Виталий Липатов,
ALT Linux Team

[devel] Re: Вмешательство в пакеты других мантейнеров

[Sergey V Turchin]

On Monday, 21 August 2023 22:55:45 MSK Vitaly Lipatov wrote:
> Sergey V Turchin писал(а) 15.8.23 11:35:
> ...
> 
> >> Так мы здесь и формулируем общепринятую трактовку.
> > 
> > "Отучаемся говорить за всех" * 2
> 
> Если вы покинули Team, это не значит, что теперь ваше отдельное мнение
> что-то значит.
Не покинул, т.е. по вашему значит.
Ваше не имеет значения при наличии _готового_ и _работающего_ механизма 
регулирования этой проблемы. И без необходимости плодить ещё один полиси.

-- 
Regards, Sergey.