[devel] I: branch release naming policy violation

[devel] I: branch release naming policy violation

[Dmitry V. Levin]

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

On Wed, Feb 04, 2009 at 12:57:25PM +0300, QA Team Robot wrote:
[...]
> openmpi - A powerful implementaion of MPI
> * Wed Feb 04 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2.M50.1
> - build for branch/5.0
> * Sun Feb 01 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2

Это противоречит букве и духу именования релизов в бранче.

Уже не первое нарушение, придётся срочно кодить проверку релизов.


-- 
ldv

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

Re: [devel] I: branch release naming policy violation

[Stanislav Ievlev]

On Wed, Feb 04, 2009 at 01:07:47PM +0300, Dmitry V. Levin wrote:
> On Wed, Feb 04, 2009 at 12:57:25PM +0300, QA Team Robot wrote:
> [...]
> > openmpi - A powerful implementaion of MPI
> > * Wed Feb 04 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2.M50.1
> > - build for branch/5.0
> > * Sun Feb 01 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2
> 
> Это противоречит букве и духу именования релизов в бранче.
> 
> Уже не первое нарушение, придётся срочно кодить проверку релизов.
А ещё лучше сделать робота, который бы пересобирал пакет специально для бранча ;)

Re: [devel] I: branch release naming policy violation

[Dmitry V. Levin]

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

On Wed, Feb 04, 2009 at 06:44:27PM +0300, Stanislav Ievlev wrote:
> On Wed, Feb 04, 2009 at 01:07:47PM +0300, Dmitry V. Levin wrote:
> > On Wed, Feb 04, 2009 at 12:57:25PM +0300, QA Team Robot wrote:
> > [...]
> > > openmpi - A powerful implementaion of MPI
> > > * Wed Feb 04 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2.M50.1
> > > - build for branch/5.0
> > > * Sun Feb 01 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2
> > 
> > Это противоречит букве и духу именования релизов в бранче.
> > 
> > Уже не первое нарушение, придётся срочно кодить проверку релизов.
> А ещё лучше сделать робота, который бы пересобирал пакет специально для бранча ;)

Да, если робот будет пересобирать для бранча случайно, то ничего хорошего
из этой затеи не выйдет. ;)


-- 
ldv

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

Re: [devel] I: branch release naming policy violation

[Ivan A. Melnikov]

On Wed, 4 Feb 2009 18:44:27 +0300
Stanislav Ievlev <inger@altlinux.org> wrote:
> > 
> > Уже не первое нарушение, придётся срочно кодить проверку релизов.
> А ещё лучше сделать робота, который бы пересобирал пакет специально
> для бранча ;)
> 

Может, лучше для начала какой-нибудь скрипт, правящий спек, в духе
add_changelog? Чтобы сразу меняло на "правилный" релиз, и в changelog
писало стандартную фразу... 

-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] I: branch release naming policy violation

[Igor Vlasenko]

On Wed, Feb 04, 2009 at 07:49:29PM +0300, Ivan A. Melnikov wrote:
> On Wed, 4 Feb 2009 18:44:27 +0300
> Stanislav Ievlev <inger@altlinux.org> wrote:
> > > 
> > > Уже не первое нарушение, придётся срочно кодить проверку релизов.
> > А ещё лучше сделать робота, который бы пересобирал пакет специально
> > для бранча ;)
> > 
> 
> Может, лучше для начала какой-нибудь скрипт, правящий спек, в духе
> add_changelog? Чтобы сразу меняло на "правилный" релиз, и в changelog
> писало стандартную фразу... 

rpmbph <спек>.

Если же хотите скрипт, который то же делает с srpm
что-то вроде
srpmbackport <src.rpm>
то могу написать за 5 мин.
Основное препятствие - придумать нормальное название :)


-- 

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

Re: [devel] I: branch release naming policy violation

[Denis Pynkin]

On 4 February 2009, Dmitry V. Levin wrote:

> > openmpi - A powerful implementaion of MPI
> > * Wed Feb 04 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2.M50.1
> > - build for branch/5.0
вы его зарезали ? 

> > * Sun Feb 01 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2
> Это противоречит букве и духу именования релизов в бранче.
угу :-( исправляюсь

кстати, начал было шерстить wiki по поводу буквы именования - к своему стыду 
могу признать, что с трудом отыскал, да и то - только в бэкпортс полиси:
http://www.altlinux.org/BackportsPolicy

<offtopic>
блин, я знал что где-то засада ;-)
не в ту сторону приоритет сделал
</offtopic>

-- 
wbr, d4s

Re: [devel] I: branch release naming policy violation

[Andrey Rahmatullin]

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

On Wed, Feb 04, 2009 at 08:37:48PM +0200, Denis Pynkin wrote:
> могу признать, что с трудом отыскал, да и то - только в бэкпортс полиси:
"только"??
Это первичный источник, разве нет?

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

<Lost> thresh: читаю каменты к той баге. нашел одну умную мысль. Смотрю - ее ж
       я и написал :)
<Lost> приятно знать что моя версия от первого августа согласна с моей версией
       от 20 февраля :)

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

Re: [devel] I: branch release naming policy violation

[Ivan A. Melnikov]

On Wed, 4 Feb 2009 20:37:48 +0200
Denis Pynkin <dans@altlinux.ru> wrote:
> 
> кстати, начал было шерстить wiki по поводу буквы именования - к
> своему стыду могу признать, что с трудом отыскал, да и то - только в
> бэкпортс полиси: http://www.altlinux.org/BackportsPolicy
> 

[процитирую]

Релизы нумеруются следующим образом:
ORIG_RELEASE.DISTRO.BACKPORT_RELEASE. Таким образом, полное
наименование пакета будет таким:

%name-%version-ORIG_RELEASE.DISTRO.BACKPORT_RELEASE

К примеру, первый бэкпорт пакета foo-1.0-alt1 на branch/4.0 будет
выглядеть как foo-1.0-alt0.M40.1. 

[конец цитаты]

К сожалению, я совершенно не способен понять, как этот пример
укладывается в приведённое перед ним правило. Я совсем заработался или
в тексте policy что-то не так?

-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] I: branch release naming policy violation

[Denis Pynkin]

On 4 February 2009, Andrey Rahmatullin wrote:

> > могу признать, что с трудом отыскал, да и то - только в бэкпортс полиси:
> "только"??
> Это первичный источник, разве нет?

угу
только не то, чтобы очевидный в данном случае. имхо.
+ зная, что искать, все равно нашлось не сразу - backports и branch немного 
названиями отличаются ;-)

я это к тому, что надо бы повесить куда-то ссылку на более видное место, вот 
только куда лучше ? 

PS ради интереса, представьте на минуту, что вы не знаете о backports policy и 
попытайтесь на wiki найти информацию о правилах именования пакетов в 
бранчах ;-)

-- 
wbr, d4s

Re: [devel] I: branch release naming policy violation

[Igor Vlasenko]

On Wed, Feb 04, 2009 at 09:53:12PM +0300, Ivan A. Melnikov wrote:
> %name-%version-ORIG_RELEASE.DISTRO.BACKPORT_RELEASE
> К сожалению, я совершенно не способен понять, как этот пример
> укладывается в приведённое перед ним правило. Я совсем заработался или
> в тексте policy что-то не так?

Да, ORIG_RELEASE -- явно выбран неправильный термин.

Я сейчас добавлю разьяснение.

-- 

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

Re: [devel] I: branch release naming policy violation

[Igor Vlasenko]

On Wed, Feb 04, 2009 at 09:29:38PM +0200, Igor Vlasenko wrote:
> On Wed, Feb 04, 2009 at 09:53:12PM +0300, Ivan A. Melnikov wrote:
> > %name-%version-ORIG_RELEASE.DISTRO.BACKPORT_RELEASE
> > К сожалению, я совершенно не способен понять, как этот пример
> > укладывается в приведённое перед ним правило. Я совсем заработался или
> > в тексте policy что-то не так?
> 
> Да, ORIG_RELEASE -- явно выбран неправильный термин.
> Я сейчас добавлю разьяснение.

например, такой вариант.
Кто хочет уточнить, правьте wiki.

Релизы нумеруются следующим образом: ADAPTED_RELEASE.DISTRO.BACKPORT_RELEASE. Таким образом, полное наименование пакета будет таким: %name-%version-ADAPTED_RELEASE.DISTRO.BACKPORT_RELEASE

К примеру, первый бэкпорт пакета foo-1.0-alt1 на branch/4.0 будет выглядеть как foo-1.0-alt0.M40.1.

Где:

    * ADAPTED_RELEASE -- релиз, выбраный таким образом, что ORIG_RELEASE <= ADAPTED_RELEASE < SISYPHUS_RELEASE. 

    Рекомендуемое значение -- SISYPHUS_RELEASE - 1. 

    * ORIG_RELEASE  строка, описывающая релиз пакета, из которого растет данная ветка (который был переложен в бранч из Сизифа);
    * SISYPHUS_RELEASE  релиз пакета в Сизифе, на базе которого делался backport пакет;
    * BACKPORT_RELEASE  номер релиза пакета внутри репозитория backports. Нумерация начинается с 1.
    * DISTRO  дистрибутив, на который осуществляется портирование.

-- 

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

Re: [devel] I: branch release naming policy violation

[Dmitry V. Levin]

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

On Wed, Feb 04, 2009 at 09:04:01PM +0200, Denis Pynkin wrote:
> On 4 February 2009, Andrey Rahmatullin wrote:
> 
> > > могу признать, что с трудом отыскал, да и то - только в бэкпортс полиси:
> > "только"??
> > Это первичный источник, разве нет?
> 
> угу
> только не то, чтобы очевидный в данном случае. имхо.
> + зная, что искать, все равно нашлось не сразу - backports и branch немного 
> названиями отличаются ;-)

Дело тут вот в чём: backports policy -- это не правило формирования
backports, а правило портирования пакетов.


-- 
ldv

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

Re: [devel] I: branch release naming policy violation

[Sergey V Turchin]

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

On Wednesday 04 February 2009, Stanislav Ievlev wrote:

[...]
> А ещё лучше сделать робота, который бы пересобирал пакет
> специально для бранча ;)
+1
#16627

-- 
Regards, Sergey, ALT Linux Team, http://www.altlinux.ru
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] I: branch release naming policy violation

[Vladimir V. Kamarzin]

>>>>> On 04 Feb 2009 at 22:28 "IV" == Igor Vlasenko writes:

>> > > Уже не первое нарушение, придётся срочно кодить проверку релизов.
>> > А ещё лучше сделать робота, который бы пересобирал пакет специально
>> > для бранча ;)
>> > 
>> 
>> Может, лучше для начала какой-нибудь скрипт, правящий спек, в духе
>> add_changelog? Чтобы сразу меняло на "правилный" релиз, и в changelog
>> писало стандартную фразу... 

IV> rpmbph <спек>.

Этот скрипт пытается делать больше того, чего от него хочется.

% rpmbph -n -M40 bugzilla.spec
Просто упаковывается bugzilla-3.2.1-alt0.M40.1.src.rpm
Платформы для сборки: i586
Сборка для платформы i586
ошибка: Файл /home/vvk/RPM/SOURCES/bugzilla-3.2.1.tar.gz: Нет такого файла или каталога
Error: Error with rpmbuild
Error: Can't build

-- 
vvk

Re: [devel] I: branch release naming policy violation

[Ivan A. Melnikov]

On Wed, 4 Feb 2009 21:40:52 +0200
Igor Vlasenko <vlasenko@imath.kiev.ua> wrote:
[...]
>     * ADAPTED_RELEASE -- релиз, выбраный таким образом, что
> ORIG_RELEASE <= ADAPTED_RELEASE < SISYPHUS_RELEASE. 
>
>     Рекомендуемое значение -- SISYPHUS_RELEASE - 1.
[...]
>     * SISYPHUS_RELEASE  релиз пакета в Сизифе, на базе которого
> делался backport пакет;

Меня немного смутило, что в SISYPHUS_RELEASE имеет вид altN (строка),
и из неё приходится вычитать единицу. С другой стороны, моя попытка
написать более корректную формулировку привела к такому запутанному
тексту, что у меня пропало всякое желание придираться к мелочам ;)

-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] I: branch release naming policy violation

[Slava Dubrovskiy]

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

Dmitry V. Levin пишет:
>> + зная, что искать, все равно нашлось не сразу - backports и branch немного 
>> названиями отличаются ;-)
>>     
>
> Дело тут вот в чём: backports policy -- это не правило формирования
> backports, а правило портирования пакетов.
>   
А где находятся правила формирования backports и branch?
Например я правильно понимаю, что если я хочу поместить пакет в бранч,
то сначала я должен поместить пакет в Сизиф?

-- 
WBR,
Dubrovskiy Vyacheslav


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3262 bytes --]

Re: [devel] I: branch release naming policy violation

[Alexey I. Froloff]

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

* Slava Dubrovskiy <slava@> [090205 13:07]:
> >> + зная, что искать, все равно нашлось не сразу - backports и branch немного 
> >> названиями отличаются ;-)
> > Дело тут вот в чём: backports policy -- это не правило формирования
> > backports, а правило портирования пакетов.
> А где находятся правила формирования backports и branch?
> Например я правильно понимаю, что если я хочу поместить пакет в бранч,
> то сначала я должен поместить пакет в Сизиф?
Кстати, да.  Надо бы сделать на сборочнице проверку, что версия
пакета в branch должна быть не больше версии этого же пакета в
сизифе.  Чтобы сбэкпорченый (собранный в среде branch) пакет
гарантированно обновился при обновлении до сизифа (или следующего
бранча, когда 5.1 появится).

Соответственно совсем новые пакеты в бранчи должны попадать через
сизиф.

-- 
Regards,
Sir Raorn.

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

Re: [devel] I: branch release naming policy violation

[Slava Dubrovskiy]

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

Alexey I. Froloff пишет:
> * Slava Dubrovskiy <slava@> [090205 13:07]:
>   
>>>> + зная, что искать, все равно нашлось не сразу - backports и branch немного 
>>>> названиями отличаются ;-)
>>>>         
>>> Дело тут вот в чём: backports policy -- это не правило формирования
>>> backports, а правило портирования пакетов.
>>>       
>> А где находятся правила формирования backports и branch?
>> Например я правильно понимаю, что если я хочу поместить пакет в бранч,
>> то сначала я должен поместить пакет в Сизиф?
>>     
> Кстати, да.  Надо бы сделать на сборочнице проверку, что версия
> пакета в branch должна быть не больше версии этого же пакета в
> сизифе.  Чтобы сбэкпорченый (собранный в среде branch) пакет
> гарантированно обновился при обновлении до сизифа (или следующего
> бранча, когда 5.1 появится).
>
> Соответственно совсем новые пакеты в бранчи должны попадать через
> сизиф.
>   
Должны, но они не попадают. Например, такой пакет как
kernel-image-ovz-rhel-2.6.18-alt2.M40.2 попал в бранч и не попал в сизиф.

-- 
WBR,
Dubrovskiy Vyacheslav


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3262 bytes --]

Re: [devel] I: branch release naming policy violation

[Денис Смирнов]

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

On Wed, Feb 04, 2009 at 07:28:04PM +0200, Igor Vlasenko wrote:

IV> Если же хотите скрипт, который то же делает с srpm
IV> что-то вроде
IV> srpmbackport <src.rpm>
IV> то могу написать за 5 мин.
IV> Основное препятствие - придумать нормальное название :)

Думаю лучше будет написать патч к rpmbph, чтобы он поддерживал еще и
готовые srpm а не только спеки. И отдать патч Виталию. По моему опыту 99%
что патч будет приложен очень быстро :)

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

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] I: branch release naming policy violation

[Денис Смирнов]

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

On Thu, Feb 05, 2009 at 12:10:24PM +0500, Vladimir V. Kamarzin wrote:

VVK> Этот скрипт пытается делать больше того, чего от него хочется.
VVK> % rpmbph -n -M40 bugzilla.spec
VVK> Просто упаковывается bugzilla-3.2.1-alt0.M40.1.src.rpm
VVK> Платформы для сборки: i586
VVK> Сборка для платформы i586
VVK> ошибка: Файл /home/vvk/RPM/SOURCES/bugzilla-3.2.1.tar.gz: Нет такого файла или каталога
VVK> Error: Error with rpmbuild
VVK> Error: Can't build

Думаю нужен отдельно ключик "патчить спек и ничего больше не делать". И
этот ключик ставить автоматом если мы находимся в git-repo.

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

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] I: branch release naming policy violation

[Eugene Prokopiev]

>  Думаю нужен отдельно ключик "патчить спек и ничего больше не делать". И
>  этот ключик ставить автоматом если мы находимся в git-repo.

+1

-- 
С уважением,
Прокопьев Евгений

Re: [devel] I: branch release naming policy violation

[Igor Vlasenko]

On Thu, Feb 05, 2009 at 03:53:18PM +0300, Денис Смирнов wrote:
> IV> Если же хотите скрипт, который то же делает с srpm
> IV> что-то вроде
> IV> srpmbackport <src.rpm>
> IV> то могу написать за 5 мин.
> IV> Основное препятствие - придумать нормальное название :)
> Думаю лучше будет написать патч к rpmbph, чтобы он поддерживал еще и
> готовые srpm а не только спеки. И отдать патч Виталию. По моему опыту 99%
> что патч будет приложен очень быстро :)

language mismatch :(

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

В sh-скрипт интегрировать такое хозяйство проблематично.

написал и выложил srpmbackport 
c пакетом perl-RPM-Source-Editor.

Это не точная замена rpmbph, 
но при необходимости проще будет переписать
логику rpmbph на перле как hook к srpmbackport.

-- 

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

Re: [devel] I: branch release naming policy violation

[Ivan Fedorov]

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

Slava Dubrovskiy <slava-caBWNg7LwhOlzlObeVyAOA@public.gmane.org> writes:

>> Соответственно совсем новые пакеты в бранчи должны попадать через
>> сизиф.
>>   
> Должны, но они не попадают. Например, такой пакет как
> kernel-image-ovz-rhel-2.6.18-alt2.M40.2 попал в бранч и не попал в сизиф.
Ну он видимо и работать нынче в сизифе не будет...

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

Re: [devel] I: branch release naming policy violation

[Dmitry V. Levin]

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

On Wed, Feb 04, 2009 at 01:07:47PM +0300, Dmitry V. Levin wrote:
> On Wed, Feb 04, 2009 at 12:57:25PM +0300, QA Team Robot wrote:
> [...]
> > openmpi - A powerful implementaion of MPI
> > * Wed Feb 04 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2.M50.1
> > - build for branch/5.0
> > * Sun Feb 01 2009 Denis Pynkin <dans@altlinux> 1.3.0-alt2
> 
> Это противоречит букве и духу именования релизов в бранче.
> 
> Уже не первое нарушение, придётся срочно кодить проверку релизов.

Реализованы следующие 2 проверки пакетов, отправляемых на _сборку_:

1. проверка имени релиза:
  - релиз пакета, собираемого в 5.0, должен иметь суффикс .M50.*
  - релиз пакета, собираемого в sisyphus, должен не иметь такого суффикса.
2. проверка версии-релиза в сизифе и бранче:
  - версия-релиз пакета, собираемого в 5.0, должна быть меньше, чем
    версия-релиз одноимённого пакета в Сизифе, если там этот пакет уже есть.
  - версия-релиз пакета, собираемого в Сизиф, должна быть больше, чем
    версия-релиз одноимённого пакета в 5.0, если там этот пакет уже есть.


-- 
ldv

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