Re: [devel] [#223227] DONE srpm=perl-Math-Pari-2.030507-alt1_2.src.rpm

Re: [devel] [#223227] DONE srpm=perl-Math-Pari-2.030507-alt1_2.src.rpm

[Dmitry V. Levin]

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

On Thu, Feb 28, 2019 at 12:52:49PM +0000, Girar Builder pender robot wrote:
> http://git.altlinux.org/tasks/archive/done/_217/223227/logs/events.1.4.log
> 
> 2019-Feb-28 12:43:23 :: task #223227 for sisyphus started by viy:
> #100 build perl-Math-Pari-2.030507-alt1_2.src.rpm
[...]
> 2019-Feb-28 12:52:11 :: updated /srpms/p/perl-Math-Pari.git branch `sisyphus'
> 2019-Feb-28 12:52:48 :: gears update OK
> 2019-Feb-28 12:52:48 :: task #223227 for sisyphus DONE

Могу констатировать: в результате перехода этого пакета на fedora import
файл perl-Math-Pari.spec существенно испортился.
Подозреваю, что и результат тоже пострадал.


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] [#223227] DONE srpm=perl-Math-Pari-2.030507-alt1_2.src.rpm

[Igor Vlasenko]

On Thu, Feb 28, 2019 at 04:01:14PM +0300, Dmitry V. Levin wrote:
> Могу констатировать: в результате перехода этого пакета на fedora import
> файл perl-Math-Pari.spec существенно испортился.
> Подозреваю, что и результат тоже пострадал.

Зря. В итоге пакет существенно выпрямился. Ранее он таскал с собой
исходники библиотеки, которые достаточно криво пытался 
сконфигурировать и собрать.
Теперь библиотека собирается отдельно и прямо,
модуль с ней так же прямо линкуется.

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

> файл perl-Math-Pari.spec существенно испортился.

Во первых, нам нужны шашечки ли ехать?

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

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

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

-- 

I V

Re: [devel] [#223227] DONE srpm=perl-Math-Pari-2.030507-alt1_2.src.rpm

[Dmitry V. Levin]

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

On Thu, Feb 28, 2019 at 07:25:53PM +0200, Igor Vlasenko wrote:
> On Thu, Feb 28, 2019 at 04:01:14PM +0300, Dmitry V. Levin wrote:
> > Могу констатировать: в результате перехода этого пакета на fedora import
> > файл perl-Math-Pari.spec существенно испортился.
> > Подозреваю, что и результат тоже пострадал.
> 
> Зря. В итоге пакет существенно выпрямился. Ранее он таскал с собой
> исходники библиотеки, которые достаточно криво пытался 
> сконфигурировать и собрать.
> Теперь библиотека собирается отдельно и прямо,
> модуль с ней так же прямо линкуется.
> 
> Я достаточно долго пытался обновить его в старом кривом виде,
> но в итоге плюнул и заменил на импорт.
> 
> > файл perl-Math-Pari.spec существенно испортился.
> 
> Во первых, нам нужны шашечки ли ехать?

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

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

Импорт в Сизиф файлов с таким содержанием как в perl-Math-Pari.spec --
это воплощение карго-культа:

http://git.altlinux.org/srpms/p/..git?p=perl-Math-Pari.git;a=blob;f=perl-Math-Pari.spec


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] [#223227] DONE srpm=perl-Math-Pari-2.030507-alt1_2.src.rpm

[Igor Vlasenko]

On Thu, Feb 28, 2019 at 08:49:30PM +0300, Dmitry V. Levin wrote:
> Импорт в Сизиф файлов с таким содержанием как в perl-Math-Pari.spec --
> это воплощение карго-культа:
> 
> http://git.altlinux.org/srpms/p/..git?p=perl-Math-Pari.git;a=blob;f=perl-Math-Pari.spec

Гм. а что там страшного с содержанием?

Единственное, что мне попалось на глаза -
скрипт не справился с преобразованием
%global __provides_exclude ^(perl\\(Math::Pari\\)$|Pari\\.so)
заменил
%filter_from_provides /^\(perl(Math.Pari\\)$\|Pari\\.so\)/d
а правильно было бы
%filter_from_provides /^\(perl(Math.Pari.pm\\)$\|Pari\\.so\)/d
Осталась висеть в спеке сопля. 
В принципе, не страшно, 
поскольку фильтр в данном случае по хорошему и не нужен.

Но раз раздражает, я убрал -
#223288 BUILDING #1 [locked] sisyphus srpm=perl-Math-Pari-2.030507-alt2_2.src.rpm

IMHO, наоборот, показывает крутость преобразователя,
который замечает даже такие мелочи, как __provides_exclude
и транслирует.

-- 

I V