ALT Linux Team development discussions
 help / color / mirror / Atom feed
* Re: [devel] СБорка perl
@ 2010-11-04 11:27 Igor Vlasenko
  2010-11-04 11:32 ` Igor Vlasenko
  2010-11-04 11:33 ` Денис Смирнов
  0 siblings, 2 replies; 20+ messages in thread
From: Igor Vlasenko @ 2010-11-04 11:27 UTC (permalink / raw)
  To: devel

On Thu, Nov 04, 2010 at 11:58:16AM +0300, Vladimir Lettiev wrote:
> Большое спасибо, Игорь, это здорово поможет. Вчера не успел добраться
> до почты. Думаю сегодня можно начать работу по формированию таска.

Сразу извиняюсь, поотлаживал багов, поэтому прошу пользоваться
girar-nmu-0.14-alt1, взять можно у меня в .git или в task #31973.
Также надо взять достаточно свежий perl-RPM-Source-Editor из сизифа.
для гарантии 0.55-alt2 у меня в .git или в task #31971.

> Возникла пара вопросов:
> 1. Как узнать, собирается ли пакет из git или srpm

утилиты разбираются с этим автоматически, -- ничего делать не нужно.

для удобства пользователя в girar-nmu есть утилита
girar-get-upload-method <name>
$ girar-get-upload-method aspell 
git
$ girar-get-upload-method aspell-de
srpm

> 2. Если в спеке не указан тэг Packager
Можно, например, в hsh использовать опцию --packager:

тег Packager устарел с введением acl. Это обсуждалось в devel@,
но, к сожалению, в документации это нигде не отражено.
Надо где-то описать на wiki.
 
> Как я понимаю, сейчас права на добавление подзаданий в задание #29354
> у меня нет, надо ли создавать новый таск?

Отдельный таск со всех сторон вещь правильная.
если вы за ноутбуком /проблемы с трафиком/, заливайте в задание perl,
а я добавлю NMU генерат.

Еще: тот perl_51x_upgrade.pl
был слишком разговорчивый, в аттачменте 
perl_51x_upgrade.pl с выключенным логом.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine



^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:27 [devel] СБорка perl Igor Vlasenko
@ 2010-11-04 11:32 ` Igor Vlasenko
  2010-11-04 11:33 ` Денис Смирнов
  1 sibling, 0 replies; 20+ messages in thread
From: Igor Vlasenko @ 2010-11-04 11:32 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 388 bytes --]

On Thu, Nov 04, 2010 at 01:27:40PM +0200, Igor Vlasenko wrote:
> Еще: тот perl_51x_upgrade.pl
> был слишком разговорчивый, в аттачменте 
> perl_51x_upgrade.pl с выключенным логом.

теперь прикрепил аттачмент.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine


[-- Attachment #2: perl_51x_upgrade.pl --]
[-- Type: application/x-perl, Size: 971 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:27 [devel] СБорка perl Igor Vlasenko
  2010-11-04 11:32 ` Igor Vlasenko
@ 2010-11-04 11:33 ` Денис Смирнов
  2010-11-04 11:47   ` Igor Vlasenko
  1 sibling, 1 reply; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 11:33 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 544 bytes --]

On Thu, Nov 04, 2010 at 01:27:40PM +0200, Igor Vlasenko wrote:

IV> тег Packager устарел с введением acl. Это обсуждалось в devel@,
IV> но, к сожалению, в документации это нигде не отражено.
IV> Надо где-то описать на wiki.

Некоторое время репокоп ругался на его отсутствие. Возможно стоит ему
начать ругаться на его присутствие (отсылая на страничку в wiki где
объяснено почему не надо ставить этот тэг)?

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:33 ` Денис Смирнов
@ 2010-11-04 11:47   ` Igor Vlasenko
  2010-11-04 11:50     ` Денис Смирнов
  0 siblings, 1 reply; 20+ messages in thread
From: Igor Vlasenko @ 2010-11-04 11:47 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Nov 04, 2010 at 02:33:28PM +0300, Денис Смирнов wrote:
> On Thu, Nov 04, 2010 at 01:27:40PM +0200, Igor Vlasenko wrote:
> 
> IV> тег Packager устарел с введением acl. Это обсуждалось в devel@,
> IV> но, к сожалению, в документации это нигде не отражено.
> IV> Надо где-то описать на wiki.
> 
> Некоторое время репокоп ругался на его отсутствие. Возможно стоит ему
> начать ругаться на его присутствие (отсылая на страничку в wiki где
> объяснено почему не надо ставить этот тэг)?

Ранее Packager был рекомендуемым тегом. Думаю, это слишком радикально,
повороты на 180". Предлагаю для начала убрать как устаревшую фразу 
из NMU policy [ http://www.altlinux.org/NMU ]

"Если в spec-файле отсутствует поле Packager, 
то его необходимо добавить и указать в нём мейнтейнера пакета."

на

"тег Packager устарел с введением acl, соответственно
манипуляции с тегом Packager в предыдущей редакции этого полиси
устарели и их выполнение не требуется."


-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine



^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:47   ` Igor Vlasenko
@ 2010-11-04 11:50     ` Денис Смирнов
  2010-11-04 11:55       ` Igor Vlasenko
  0 siblings, 1 reply; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 11:50 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 1147 bytes --]

On Thu, Nov 04, 2010 at 01:47:11PM +0200, Igor Vlasenko wrote:

IV> Ранее Packager был рекомендуемым тегом. Думаю, это слишком радикально,
IV> повороты на 180". Предлагаю для начала убрать как устаревшую фразу 
IV> из NMU policy [ http://www.altlinux.org/NMU ]
IV> "Если в spec-файле отсутствует поле Packager, 
IV> то его необходимо добавить и указать в нём мейнтейнера пакета."
IV> на
IV> "тег Packager устарел с введением acl, соответственно
IV> манипуляции с тегом Packager в предыдущей редакции этого полиси
IV> устарели и их выполнение не требуется."

Вопрос -- автоматом проставляется какой Packager? Тот, кто собрал пакет,
или тот кто является лидером группы?

Если первое -- то мы имеем проблему (я делаю NMU твоего пакета, и в поле
Packager будет 'mithraen', что неправда и делает использование поля
Packager бессмысленным).

Если же при сборке пакета поле Packager выставляется в соответствии с
именем лидера в acl пакета, то тогда вручную прописанное поле Packager
становится вредным.

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:50     ` Денис Смирнов
@ 2010-11-04 11:55       ` Igor Vlasenko
  2010-11-04 12:14         ` Денис Смирнов
                           ` (2 more replies)
  0 siblings, 3 replies; 20+ messages in thread
From: Igor Vlasenko @ 2010-11-04 11:55 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Nov 04, 2010 at 02:50:16PM +0300, Денис Смирнов wrote:
> On Thu, Nov 04, 2010 at 01:47:11PM +0200, Igor Vlasenko wrote:
> 
> IV> Ранее Packager был рекомендуемым тегом. Думаю, это слишком радикально,
> IV> повороты на 180". Предлагаю для начала убрать как устаревшую фразу 
> IV> из NMU policy [ http://www.altlinux.org/NMU ]
> IV> "Если в spec-файле отсутствует поле Packager, 
> IV> то его необходимо добавить и указать в нём мейнтейнера пакета."
> IV> на
> IV> "тег Packager устарел с введением acl, соответственно
> IV> манипуляции с тегом Packager в предыдущей редакции этого полиси
> IV> устарели и их выполнение не требуется."
> 
> Вопрос -- автоматом проставляется какой Packager? Тот, кто собрал пакет,
как помню, да.
> или тот кто является лидером группы?

 
> Если первое -- то мы имеем проблему (я делаю NMU твоего пакета, и в поле
> Packager будет 'mithraen', что неправда и делает использование поля
> Packager бессмысленным).

поэтому в старом nmu полиси и была такая рекомендация.
Но проблема в том, что сейчас много пакетов, у которых в Packager
записаны люди, которых нет и в acl.

Получается, либо при смене acl автоматом пересобирать src.rpm,
либо отказаться от опоры на Packager, как это сделал 
Игорь Зубков в prometeus2.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine



^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:55       ` Igor Vlasenko
@ 2010-11-04 12:14         ` Денис Смирнов
  2010-11-04 12:17         ` Vladislav Zavjalov
  2010-11-04 12:18         ` Igor Zubkov
  2 siblings, 0 replies; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 12:14 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 759 bytes --]

On Thu, Nov 04, 2010 at 01:55:49PM +0200, Igor Vlasenko wrote:

IV> Получается, либо при смене acl автоматом пересобирать src.rpm,
IV> либо отказаться от опоры на Packager, как это сделал 
IV> Игорь Зубков в prometeus2.

В наших скриптах и сервисах -- да, надо. Но пользователи таки иногда
смотрят на rpm -qi. Конечно, информация о packager'е иногда может быть
неверна, но случай "мантейнер забил" в общем-то понятный, информация может
устаревать.

А вот случай 'в Packager -- чушь' непонятный и неправильный.

Так сейчас что пишется в поле Packager при сборке пакета в gear? Имя того
его собрал или имя лидера acl?

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:55       ` Igor Vlasenko
  2010-11-04 12:14         ` Денис Смирнов
@ 2010-11-04 12:17         ` Vladislav Zavjalov
  2010-11-04 13:54           ` Mikhail Efremov
  2010-11-04 12:18         ` Igor Zubkov
  2 siblings, 1 reply; 20+ messages in thread
From: Vladislav Zavjalov @ 2010-11-04 12:17 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Nov 04, 2010 at 01:55:49PM +0200, Igor Vlasenko wrote:
> Получается, либо при смене acl автоматом пересобирать src.rpm,
> либо отказаться от опоры на Packager, как это сделал 
> Игорь Зубков в prometeus2.

Наверное, надо сперва понять, какую информацию этот тэг должен нести, а
потом уже - как его заполнять. Мне вот, например, видится такая картина:

- acl - сейчас несет информацию о двух разных сущностях: кто текущий мантейнер (leader)
  и кто вообще может этот пакет пересобрать (например, очевидно, что
  у "A + everybody" и "B + everybody" различается только первая сущность,
  а у "A + everybody" и "A" - только вторая).

- changelog - кто собирал данный пакет, когда и зачем.

- packager - видимо, его смысл должен быть ближе всего к "мантейнер
  (= acl leader!) на момент сборки данного пакета". Если это так - то понятно,
  как его заполнять. И не надо ничего пересобирать.

Слава




^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 11:55       ` Igor Vlasenko
  2010-11-04 12:14         ` Денис Смирнов
  2010-11-04 12:17         ` Vladislav Zavjalov
@ 2010-11-04 12:18         ` Igor Zubkov
  2010-11-04 12:24           ` Денис Смирнов
  2 siblings, 1 reply; 20+ messages in thread
From: Igor Zubkov @ 2010-11-04 12:18 UTC (permalink / raw)
  To: ALT Linux Team development discussions

2010/11/4 Igor Vlasenko:
> поэтому в старом nmu полиси и была такая рекомендация.
> Но проблема в том, что сейчас много пакетов, у которых в Packager
> записаны люди, которых нет и в acl.
>
> Получается, либо при смене acl автоматом пересобирать src.rpm,
> либо отказаться от опоры на Packager, как это сделал
> Игорь Зубков в prometeus2.

Тут не всё так просто. Да, в prometheus2 тег Packager практически
игнорируется. Вместо него используются acl для показа нужных пакетов
на странице маинтейнера. Но из него всё таки данные извлекаются что бы
собрать таблицу с маинтейнерами. Предлагаю не заполнять поле Packager:
в спек файле руками, но что бы rpm и hasher его сами заполняли его. В
противном случае, мне придётся долго и упорно собирать таблицу с
именами и логинами руками для prometheus2.

-- 
Igor Zubkov
http://hi.im/ice

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 12:18         ` Igor Zubkov
@ 2010-11-04 12:24           ` Денис Смирнов
  0 siblings, 0 replies; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 12:24 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 836 bytes --]

On Thu, Nov 04, 2010 at 02:18:47PM +0200, Igor Zubkov wrote:

IZ> Тут не всё так просто. Да, в prometheus2 тег Packager практически
IZ> игнорируется. Вместо него используются acl для показа нужных пакетов
IZ> на странице маинтейнера. Но из него всё таки данные извлекаются что бы
IZ> собрать таблицу с маинтейнерами. Предлагаю не заполнять поле Packager:
IZ> в спек файле руками, но что бы rpm и hasher его сами заполняли его. В
IZ> противном случае, мне придётся долго и упорно собирать таблицу с
IZ> именами и логинами руками для prometheus2.

Сейчас так и есть.

Я предлагаю внести только одно изменение -- в тэг Packager должен попадать
не тот кто собрал пакет, а тот кто является лидером.

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 12:17         ` Vladislav Zavjalov
@ 2010-11-04 13:54           ` Mikhail Efremov
  2010-11-04 14:04             ` Vladislav Zavjalov
  2010-11-04 14:05             ` Денис Смирнов
  0 siblings, 2 replies; 20+ messages in thread
From: Mikhail Efremov @ 2010-11-04 13:54 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, 4 Nov 2010 15:17:58 +0300 Vladislav Zavjalov wrote:
> - acl - сейчас несет информацию о двух разных сущностях: кто текущий
> мантейнер (leader) и кто вообще может этот пакет пересобрать
> (например, очевидно, что у "A + everybody" и "B + everybody"
> различается только первая сущность, а у "A + everybody" и "A" -
> только вторая).
> 
> - changelog - кто собирал данный пакет, когда и зачем.
> 
> - packager - видимо, его смысл должен быть ближе всего к "мантейнер
>   (= acl leader!) на момент сборки данного пакета". Если это так - то
> понятно, как его заполнять. И не надо ничего пересобирать.

А зачем нужна эта информация? О том, что когда-то, на момент сборки
пакета, лидером был этот человек? Информация об acl может
устареть хоть сразу же после сборки пакета. Надеюсь, пересобирать
пакеты при изменении acl никто всерьез предлагать не будет.
Мне думается, что сейчас как раз самое разумное значение этого поля -
это как раз имя того, кто последний собирал пакет. Во всех остальных
случаях это поле не несет абсолютно никакой полезной информации. И
смотреть всегда лучше на acl и changelog пакета.
В том, что при NMU в packager будет стоять имя человека, делавшего NMU,
а не мантейнера, я ничего ужасного не вижу.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 13:54           ` Mikhail Efremov
@ 2010-11-04 14:04             ` Vladislav Zavjalov
  2010-11-04 14:16               ` Mikhail Efremov
  2010-11-04 14:05             ` Денис Смирнов
  1 sibling, 1 reply; 20+ messages in thread
From: Vladislav Zavjalov @ 2010-11-04 14:04 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Nov 04, 2010 at 04:54:20PM +0300, Mikhail Efremov wrote:
> On Thu, 4 Nov 2010 15:17:58 +0300 Vladislav Zavjalov wrote:
> > - acl - сейчас несет информацию о двух разных сущностях: кто текущий
> > мантейнер (leader) и кто вообще может этот пакет пересобрать
> > (например, очевидно, что у "A + everybody" и "B + everybody"
> > различается только первая сущность, а у "A + everybody" и "A" -
> > только вторая).
> > 
> > - changelog - кто собирал данный пакет, когда и зачем.
> > 
> > - packager - видимо, его смысл должен быть ближе всего к "мантейнер
> >   (= acl leader!) на момент сборки данного пакета". Если это так - то
> > понятно, как его заполнять. И не надо ничего пересобирать.
> 
> А зачем нужна эта информация? О том, что когда-то, на момент сборки
> пакета, лидером был этот человек? Информация об acl может
> устареть хоть сразу же после сборки пакета.

Ну, у тебя есть некоторый объект. В объекте хранится информация о том,
кто отвечал за его создание - на тот момент, в тех условиях, в которых
он собирался.
А если ты хочешь посмотреть, кто сейчас отвечает за создание подобных
объектов, например, в Сизифе - ты смотришь сизифные acl.

> Мне думается, что сейчас как раз самое разумное значение этого поля -
> это как раз имя того, кто последний собирал пакет.

Это в точности дублирование changelog. Зачем?

Слава


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 13:54           ` Mikhail Efremov
  2010-11-04 14:04             ` Vladislav Zavjalov
@ 2010-11-04 14:05             ` Денис Смирнов
  2010-11-04 14:21               ` Mikhail Efremov
  2010-11-08  7:25               ` Vladimir V. Kamarzin
  1 sibling, 2 replies; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 14:05 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]

On Thu, Nov 04, 2010 at 04:54:20PM +0300, Mikhail Efremov wrote:

ME> А зачем нужна эта информация? О том, что когда-то, на момент сборки
ME> пакета, лидером был этот человек? Информация об acl может

На момент сборки этого пакета, ответственным за его качество был такой-то.

Это единственный реальный смысл поля Packager в rpm, который я могу себе
представить.

ME> Мне думается, что сейчас как раз самое разумное значение этого поля -
ME> это как раз имя того, кто последний собирал пакет. Во всех остальных
ME> случаях это поле не несет абсолютно никакой полезной информации. И
ME> смотреть всегда лучше на acl и changelog пакета.

Имя того кто последний собрал пакет у нас есть -- в первой строчке
changelog.

ME> В том, что при NMU в packager будет стоять имя человека, делавшего NMU,
ME> а не мантейнера, я ничего ужасного не вижу.

Это может быть, к примеру, робот. 

Главный вопрос -- для каких целей используется поле Packager? :)

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:04             ` Vladislav Zavjalov
@ 2010-11-04 14:16               ` Mikhail Efremov
  2010-11-04 19:08                 ` Денис Смирнов
  0 siblings, 1 reply; 20+ messages in thread
From: Mikhail Efremov @ 2010-11-04 14:16 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, 4 Nov 2010 17:04:11 +0300 Vladislav Zavjalov wrote:
> On Thu, Nov 04, 2010 at 04:54:20PM +0300, Mikhail Efremov wrote:
> > On Thu, 4 Nov 2010 15:17:58 +0300 Vladislav Zavjalov wrote:
> > > - acl - сейчас несет информацию о двух разных сущностях: кто
> > > текущий мантейнер (leader) и кто вообще может этот пакет
> > > пересобрать (например, очевидно, что у "A + everybody" и "B +
> > > everybody" различается только первая сущность, а у "A +
> > > everybody" и "A" - только вторая).
> > > 
> > > - changelog - кто собирал данный пакет, когда и зачем.
> > > 
> > > - packager - видимо, его смысл должен быть ближе всего к
> > > "мантейнер (= acl leader!) на момент сборки данного пакета". Если
> > > это так - то понятно, как его заполнять. И не надо ничего
> > > пересобирать.
> > 
> > А зачем нужна эта информация? О том, что когда-то, на момент сборки
> > пакета, лидером был этот человек? Информация об acl может
> > устареть хоть сразу же после сборки пакета.
> 
> Ну, у тебя есть некоторый объект. В объекте хранится информация о том,
> кто отвечал за его создание - на тот момент, в тех условиях, в которых
> он собирался.

Мне думается, что в первую очередь ответственность за изменение несет
тот, кто это изменение делает, а не тот кто стоит первым в acl. И
сейчас все равно имеет смысл смотреть только changelog, а никак не
packager.
По этому полю можно только узнать, что некто когда-то имел какое-то
отношение к пакету. Мне не кажется это сильно полезной информацией.

> > Мне думается, что сейчас как раз самое разумное значение этого поля
> > - это как раз имя того, кто последний собирал пакет.
> 
> Это в точности дублирование changelog. Зачем?

Ну да, тогда packager будет всегда совпадать с последней записью в
changelog. И да, это избыточно. Но так я вижу хоть какую-то логику в
наличии этого поля.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:05             ` Денис Смирнов
@ 2010-11-04 14:21               ` Mikhail Efremov
  2010-11-04 14:32                 ` Vladislav Zavjalov
  2010-11-04 19:12                 ` Денис Смирнов
  2010-11-08  7:25               ` Vladimir V. Kamarzin
  1 sibling, 2 replies; 20+ messages in thread
From: Mikhail Efremov @ 2010-11-04 14:21 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, 4 Nov 2010 17:05:56 +0300 Денис Смирнов wrote:
> ME> А зачем нужна эта информация? О том, что когда-то, на момент
> ME> сборки пакета, лидером был этот человек? Информация об acl может
> 
> На момент сборки этого пакета, ответственным за его качество был
> такой-то.
> 
> Это единственный реальный смысл поля Packager в rpm, который я могу
> себе представить.

При этом в данный момент этот человек может уже и не состоять в
тим. И вообще участвовать в экспедиции на Марс. Более того, это может
быть уже и на момент сборки пакета. Acl далеко не всегда меняются
оперативно.

> ME> В том, что при NMU в packager будет стоять имя человека,
> ME> делавшего NMU, а не мантейнера, я ничего ужасного не вижу.
> 
> Это может быть, к примеру, робот. 

Тоже не вижу в этом ничего ужасного.

> Главный вопрос -- для каких целей используется поле Packager? :)

Понятия не имею. Сейчас поле Packager означает только поле Packager. И
все.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:21               ` Mikhail Efremov
@ 2010-11-04 14:32                 ` Vladislav Zavjalov
  2010-11-04 16:49                   ` Mikhail Efremov
  2010-11-04 19:12                 ` Денис Смирнов
  1 sibling, 1 reply; 20+ messages in thread
From: Vladislav Zavjalov @ 2010-11-04 14:32 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Nov 04, 2010 at 05:21:49PM +0300, Mikhail Efremov wrote:
> On Thu, 4 Nov 2010 17:05:56 +0300 Денис Смирнов wrote:
> > ME> А зачем нужна эта информация? О том, что когда-то, на момент
> > ME> сборки пакета, лидером был этот человек? Информация об acl может
> > 
> > На момент сборки этого пакета, ответственным за его качество был
> > такой-то.
> > 
> > Это единственный реальный смысл поля Packager в rpm, который я могу
> > себе представить.
> 
> При этом в данный момент этот человек может уже и не состоять в
> тим. И вообще участвовать в экспедиции на Марс. Более того, это может
> быть уже и на момент сборки пакета. Acl далеко не всегда меняются
> оперативно.

Кажется, ты смешиваешь два вопроса:

- Нужно ли понятие "ответственный за пакет"? По факту - есть acl leader.
  Иногда лучше ссылаться на него, чем на разных роботов, дублируя, к тому
  же, changelog.

- Нужно ли править историю, если ответственный изменился? Мне вот
  очевидно, что нет. При избрании нового царя не нужно менять имя в
  старых указах, даже если они по-прежнему действуют, а старый царь
  уже летит на Марс...

Слава


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:32                 ` Vladislav Zavjalov
@ 2010-11-04 16:49                   ` Mikhail Efremov
  0 siblings, 0 replies; 20+ messages in thread
From: Mikhail Efremov @ 2010-11-04 16:49 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, 4 Nov 2010 17:32:17 +0300 Vladislav Zavjalov wrote:
> > > На момент сборки этого пакета, ответственным за его качество был
> > > такой-то.
> > > 
> > > Это единственный реальный смысл поля Packager в rpm, который я
> > > могу себе представить.
> > 
> > При этом в данный момент этот человек может уже и не состоять в
> > тим. И вообще участвовать в экспедиции на Марс. Более того, это
> > может быть уже и на момент сборки пакета. Acl далеко не всегда
> > меняются оперативно.
> 
> Кажется, ты смешиваешь два вопроса:
> 
> - Нужно ли понятие "ответственный за пакет"? По факту - есть acl
> leader. Иногда лучше ссылаться на него, чем на разных роботов,
> дублируя, к тому же, changelog.

Хм, ну, пожалуй, все-таки, да :).
Вообще, главное, чтобы packager содержал какую-то осмысленную
информацию. И не было необходимости указывать его значение в спеке
вручную. Текущий вариант, когда подставляется имя того, кто собирает
пакет, меня вполне устраивает. Ну пускай подставляется лидер из acl,
главное чтобы не непонятно какое имя из спека.
Так что из своих спеков я и дальше буду стараться Packager выкидывать.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:16               ` Mikhail Efremov
@ 2010-11-04 19:08                 ` Денис Смирнов
  0 siblings, 0 replies; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 19:08 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

On Thu, Nov 04, 2010 at 05:16:47PM +0300, Mikhail Efremov wrote:

ME> Мне думается, что в первую очередь ответственность за изменение несет
ME> тот, кто это изменение делает, а не тот кто стоит первым в acl. И
ME> сейчас все равно имеет смысл смотреть только changelog, а никак не
ME> packager.

Ответственность за изменение несет тот, кто указан в changelog.
Ответственность за пакет имеет тот, кто указан в Packager. По крайней мере
это человек, которого можно спросить "а на кой этот пакет собирается вот
так, а не иначе"

ME> По этому полю можно только узнать, что некто когда-то имел какое-то
ME> отношение к пакету. Мне не кажется это сильно полезной информацией.

Это информация о том, что в тот момент именно этот человек не просто имел
отношение к пакету, а имел неограниченные полномочия в принятии решений по
этому пакету. 

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:21               ` Mikhail Efremov
  2010-11-04 14:32                 ` Vladislav Zavjalov
@ 2010-11-04 19:12                 ` Денис Смирнов
  1 sibling, 0 replies; 20+ messages in thread
From: Денис Смирнов @ 2010-11-04 19:12 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 1535 bytes --]

On Thu, Nov 04, 2010 at 05:21:49PM +0300, Mikhail Efremov wrote:

ME> При этом в данный момент этот человек может уже и не состоять в
ME> тим. И вообще участвовать в экспедиции на Марс. Более того, это может
ME> быть уже и на момент сборки пакета. Acl далеко не всегда меняются
ME> оперативно.

Да, такое может быть. Однако эта информация _может быть_ ценной в ряде
случаев. У нас есть варианты:
- писать в Packager всякую чушь (как сейчас);
- писать туда последнюю строчку из %changelog (т.е. дублировать уже
  имеющуюся информацию);
- писать туда информацию, которая представляет ценность, и очевидного
  способа получить которую на нынешний момент времени нет;

И так, что лучше?

>> Это может быть, к примеру, робот. 
ME> Тоже не вижу в этом ничего ужасного.

Packager интересен именно как человек, к которому можно попытаться
обратиться с каким-то вопросом.

>> Главный вопрос -- для каких целей используется поле Packager? :)
ME> Понятия не имею. Сейчас поле Packager означает только поле Packager. И
ME> все.

Если нет понятия зачем вообще нужно поле Packager, то любые разговоры о
том что там должно быть не имеют смысла, не так ли? И даже если там будут
просто random bytes это неважно, так?

А вот для меня важно чтобы в поле Packager был указан конкретный человек
(а не робот, к примеру), который имеет или имел когда-то понимание о том
что и как в этом пакете.

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [devel] СБорка perl
  2010-11-04 14:05             ` Денис Смирнов
  2010-11-04 14:21               ` Mikhail Efremov
@ 2010-11-08  7:25               ` Vladimir V. Kamarzin
  1 sibling, 0 replies; 20+ messages in thread
From: Vladimir V. Kamarzin @ 2010-11-08  7:25 UTC (permalink / raw)
  To: ALT Linux Team development discussions

>>>>> On 04 Nov 2010 at 19:05 "m" == mithraen  writes:

m> On Thu, Nov 04, 2010 at 04:54:20PM +0300, Mikhail Efremov wrote:
ME>> А зачем нужна эта информация? О том, что когда-то, на момент сборки
ME>> пакета, лидером был этот человек? Информация об acl может

m> На момент сборки этого пакета, ответственным за его качество был такой-то.
m> Это единственный реальный смысл поля Packager в rpm, который я могу себе
m> представить.

Благодаря такой политике у нас годами Packager-ами в пакетах числятся люди,
которые давно перестали собирать пакеты.

-- 
vvk


^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2010-11-08  7:25 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-04 11:27 [devel] СБорка perl Igor Vlasenko
2010-11-04 11:32 ` Igor Vlasenko
2010-11-04 11:33 ` Денис Смирнов
2010-11-04 11:47   ` Igor Vlasenko
2010-11-04 11:50     ` Денис Смирнов
2010-11-04 11:55       ` Igor Vlasenko
2010-11-04 12:14         ` Денис Смирнов
2010-11-04 12:17         ` Vladislav Zavjalov
2010-11-04 13:54           ` Mikhail Efremov
2010-11-04 14:04             ` Vladislav Zavjalov
2010-11-04 14:16               ` Mikhail Efremov
2010-11-04 19:08                 ` Денис Смирнов
2010-11-04 14:05             ` Денис Смирнов
2010-11-04 14:21               ` Mikhail Efremov
2010-11-04 14:32                 ` Vladislav Zavjalov
2010-11-04 16:49                   ` Mikhail Efremov
2010-11-04 19:12                 ` Денис Смирнов
2010-11-08  7:25               ` Vladimir V. Kamarzin
2010-11-04 12:18         ` Igor Zubkov
2010-11-04 12:24           ` Денис Смирнов

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