[devel] sisyphus_check: рубить с плеча не так уж безопасно.

[devel] sisyphus_check: рубить с плеча не так уж безопасно.

[Igor Vlasenko]

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

Когда я внезапно узнал из письма Алексея про
AT> sisyphus_check, которые запрещает зависимость на xorg-devel
сначала был в шоке, но потом немного отошел.
Конечно, плохо, что такие вещи делались без обсуждения и
предупреждения, но Алексей сделает из этого какие-то выводы, 
подумал я, плюс, эти изменения действительно дают небольшую 
оптимизацию. Так что человек нещепетильный пусть уступит
человеку щепетильному. Раз Алексею это важно, пусть будет.

К сожалению, кода я получил сообщеня о непересобирающихся пакетах 
и посмотрел в код sisyphus_check, шок у меня случился повторно.

В sisyphus_check Алексей запретил все пакеты вида xorg-x11-.* .

Из этих пакетов желаемую Алексеем оптимизацию даст только борьба с 
xorg-x11-devel. Остальные (B)Req: пострадали безвинно, их "вина" 
только в том, что они не угнались за переименованиями в Сизифе.

"Лечение" пострадавших пакетов заулючается в механических subst
xorg-x11-server -> xorg-server
xorg-x11-proto-devel -> xorg-proto-devel
xorg-x11-locales -> libX11-locales
и, как видим, совершенно бесполезно и идет вразрез с нашей практикой
при переименовании пакета оставлять совместимые Provides:.

Кроме того, Provides: вида xorg-x11-server, xorg-x11-proto-devel,
нужны для совместимости с другими дистрибутивами, в частности,
fedora, чтобы облегчать написание кроссдистрибутивных rpm
и уменьшать дурную работу при адаптации пакета из другого дистрибутива.

Поэтому проверке xorg-x11-.* не место в sisyphus_check,
из уважения к Алексею ее можно заменить на xorg-x11-devel.

Вот списки безвинно пострадавших (B)Req:

sqlite> select distinct requirename from rpm_requires where requirename glob 'xorg-x11-*';
xorg-x11-drv-vboxmouse
xorg-x11-drv-vboxvideo
xorg-x11-drv-video
xorg-x11-font-utils
xorg-x11-libs
xorg-x11-locales
xorg-x11-proto-devel
xorg-x11-rgb
xorg-x11-server
xorg-x11-utils
xorg-x11-xauth
xorg-x11-xnest
xorg-x11-xvfb
sqlite> select distinct buildrequirename from rpm_buildrequires where buildrequirename glob 'xorg-x11-*';
xorg-x11-bitmaps
xorg-x11-font-utils
xorg-x11-libs
xorg-x11-locales
xorg-x11-mesagl
xorg-x11-proto-devel
xorg-x11-server
xorg-x11-util-macros
xorg-x11-utils
xorg-x11-xtrans-devel
xorg-x11-xvfb


-- 

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

Re: [devel] sisyphus_check: рубить с плеча не так уж безопасно.

[REAL]

28.03.2011 15:35, Igor Vlasenko пишет:
> xorg-proto-devel

На это точно нельзя делать запрет, пока в этом пакете что-то есть или 
он не переименован с сохранением Provides. Насчёт остального - ну и 
ладно, не беда ;)

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] sisyphus_check: рубить с плеча не так уж безопасно.

[Dmitry V. Levin]

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

On Mon, Mar 28, 2011 at 11:35:38AM +0300, Igor Vlasenko wrote:
> Уважаемые коллеги,
> Уважаемый Алексей, еще раз извиняюсь, что поднимаю эту тему.
> 
> Когда я внезапно узнал из письма Алексея про
> AT> sisyphus_check, которые запрещает зависимость на xorg-devel
> сначала был в шоке, но потом немного отошел.
> Конечно, плохо, что такие вещи делались без обсуждения и
> предупреждения, но Алексей сделает из этого какие-то выводы, 
> подумал я, плюс, эти изменения действительно дают небольшую 
> оптимизацию. Так что человек нещепетильный пусть уступит
> человеку щепетильному. Раз Алексею это важно, пусть будет.
> 
> К сожалению, кода я получил сообщеня о непересобирающихся пакетах 
> и посмотрел в код sisyphus_check, шок у меня случился повторно.
> 
> В sisyphus_check Алексей запретил все пакеты вида xorg-x11-.* .
> 
> Из этих пакетов желаемую Алексеем оптимизацию даст только борьба с 
> xorg-x11-devel. Остальные (B)Req: пострадали безвинно, их "вина" 
> только в том, что они не угнались за переименованиями в Сизифе.
> 
> "Лечение" пострадавших пакетов заулючается в механических subst
> xorg-x11-server -> xorg-server
> xorg-x11-proto-devel -> xorg-proto-devel
> xorg-x11-locales -> libX11-locales
> и, как видим, совершенно бесполезно и идет вразрез с нашей практикой
> при переименовании пакета оставлять совместимые Provides:.
> 
> Кроме того, Provides: вида xorg-x11-server, xorg-x11-proto-devel,
> нужны для совместимости с другими дистрибутивами, в частности,
> fedora, чтобы облегчать написание кроссдистрибутивных rpm
> и уменьшать дурную работу при адаптации пакета из другого дистрибутива.
> 
> Поэтому проверке xorg-x11-.* не место в sisyphus_check,
> из уважения к Алексею ее можно заменить на xorg-x11-devel.

Непонятно, как Алексей мог допустить столь очевидную ошибку.
Я отправил в Сизиф sisyphus_check-0.8.21-alt1 с этим изменением.


-- 
ldv

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

Re: [devel] sisyphus_check, "you do it" и взаимоуважение

[Michael Shigorin]

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

On Mon, Mar 28, 2011 at 11:35:38AM +0300, Igor Vlasenko wrote:
> Кроме того, Provides: вида xorg-x11-server, xorg-x11-proto-devel,

Их эта проверка касается?

> Поэтому проверке xorg-x11-.* не место в sisyphus_check,
> из уважения к Алексею ее можно заменить на xorg-x11-devel.

Напомню фактический принцип совместной разработки Linux:
you need it, you do it.  Делаешь, а не принуждаешь других.

2 aen, ldv: собственно, это одна из моих основных претензий
к сложившимся практикам взаимодействия в ALT Linux -- ручкой
в виде sisyphus_check пользуется ограниченное число людей и
производимые действия, особенно предпринятые явочным порядком,
порой похожи на кнут (а то ещё и без пряника, как сейчас).

Если изменения фактических политик будут обязательны к обсуждению
и тестированию с WARNING вместо ERROR (кроме неотложных случаев,
когда ldv@ как крайний по техобеспечению проекта сочтёт нужным
включить сразу ERROR) -- возможно, получится избежать множества
излишней переписки и напрасной траты нервов/людей в будущем.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

Re: [devel] sisyphus_check, "you do it" и взаимоуважение

[Aleksey Novodvorsky]

28 марта 2011 г. 13:36 пользователь Michael Shigorin <mike@osdn.org.ua> написал:
> On Mon, Mar 28, 2011 at 11:35:38AM +0300, Igor Vlasenko wrote:
>> Кроме того, Provides: вида xorg-x11-server, xorg-x11-proto-devel,
>
> Их эта проверка касается?
>
>> Поэтому проверке xorg-x11-.* не место в sisyphus_check,
>> из уважения к Алексею ее можно заменить на xorg-x11-devel.
>
> Напомню фактический принцип совместной разработки Linux:
> you need it, you do it.  Делаешь, а не принуждаешь других.
>
> 2 aen, ldv: собственно, это одна из моих основных претензий
> к сложившимся практикам взаимодействия в ALT Linux -- ручкой
> в виде sisyphus_check пользуется ограниченное число людей и
> производимые действия, особенно предпринятые явочным порядком,
> порой похожи на кнут (а то ещё и без пряника, как сейчас).
>
> Если изменения фактических политик будут обязательны к обсуждению
> и тестированию с WARNING вместо ERROR (кроме неотложных случаев,
> когда ldv@ как крайний по техобеспечению проекта сочтёт нужным
> включить сразу ERROR) -- возможно, получится избежать множества
> излишней переписки и напрасной траты нервов/людей в будущем.

Можете написать полиси на один абзац?

Rgrds, Алексей

Re: [devel] sisyphus_check, "you do it" и взаимоуважение

[Yuri N. Sedunov]

В Пнд, 28/03/2011 в 13:46 +0400, Aleksey Novodvorsky пишет:
> 28 марта 2011 г. 13:36 пользователь Michael Shigorin <mike@osdn.org.ua> написал:
> > On Mon, Mar 28, 2011 at 11:35:38AM +0300, Igor Vlasenko wrote:
<skip>
> >
> > Если изменения фактических политик будут обязательны к обсуждению
> > и тестированию с WARNING вместо ERROR (кроме неотложных случаев,
> > когда ldv@ как крайний по техобеспечению проекта сочтёт нужным
> > включить сразу ERROR) -- возможно, получится избежать множества
> > излишней переписки и напрасной траты нервов/людей в будущем.
> 
> Можете написать полиси на один абзац?
> 
Не будете успевать полиси писать, -- давно понятно, что пора отобрать
лом у выпрямителя и ассенизатора.

-- 
Yuri N. Sedunov

Re: [devel] sisyphus_check, "you do it" и взаимоуважение

[Michael Shigorin]

On Mon, Mar 28, 2011 at 01:46:17PM +0400, Aleksey Novodvorsky wrote:
> > изменения фактических политик будут обязательны к обсуждению
> > и тестированию с WARNING вместо ERROR (кроме неотложных
> > случаев, когда ldv@ как крайний по техобеспечению проекта
> > сочтёт нужным включить сразу ERROR)
> Можете написать полиси на один абзац?

По сути уже и написал.  Следует уточнить срок тестирования --
думаю, недели достаточно.  Ну и s/крайний/ответственный/

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/