[devel] perl noarch on x86_64

[devel] perl noarch on x86_64

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

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

Эти пакеты noarch, но при этом содержат что-либо внутри /usr/lib/perl5/vendor_perl/i386-linux
мне почему-то кажется что на самом деле их нельзя считать noarch

perl-Alien-wxWidgets-0.40-alt1.noarch.rpm
perl-Apache2-SOAP-0.72-alt1.noarch.rpm
perl-Asterisk-config-0.8-alt2.noarch.rpm
perl-Asterisk-FastAGI-0.02-alt1.noarch.rpm
perl-BIND-Conf_Parser-0.95-alt1.noarch.rpm
perl-Catalyst-Action-RenderView-0.04-alt1.noarch.rpm
perl-CPANPLUS-0.84-alt1.noarch.rpm
perl-Data-ICal-0.13-alt1.noarch.rpm
perl-DIME-Tools-0.03-alt1.noarch.rpm
perl-File-Scan-ClamAV-1.8-alt1.noarch.rpm
perl-GnuPG-Interface-0.36-alt1.noarch.rpm
perl-HTML-RewriteAttributes-0.02-alt1.noarch.rpm
perl-HTTP-Server-Simple-Mason-0.09-alt1.noarch.rpm
perl-IO-AIO-Util-0.06-alt1.noarch.rpm
perl-IPC-Run-SafeHandles-0.01-alt1.noarch.rpm
perl-Kolab-1.02-alt1.noarch.rpm
perl-Kolab-Util-1.01-alt1.noarch.rpm
perl-Lingua-Preferred-0.2.4-alt1.noarch.rpm
perl-Log-Dispatch-Perl-0.03-alt1.noarch.rpm
perl-Log-TraceMessages-1.4-alt1.noarch.rpm
perl-MooseX-AttributeHelpers-0.11-alt1.noarch.rpm
perl-MooseX-MultiInitArg-0.01-alt1.noarch.rpm
perl-MPlayer-0.04-alt2.noarch.rpm
perl-Nagios-Plugin-0.27-alt1.noarch.rpm
perl-Net-IP-1.24-alt1.noarch.rpm
perl-NetxAP-0.02-alt2.noarch.rpm
perl-NNTPClient-0.37-alt1.1.noarch.rpm
perl-Object-Signature-1.05-alt1.noarch.rpm
perl-POE-Component-Generic-0.1100-alt1.noarch.rpm
perl-Test-HTTP-Server-Simple-0.06-alt1.noarch.rpm
perl-Text-vFile-asData-0.05-alt1.noarch.rpm
perl-Tie-Filehandle-Preempt-Stdin-0.01-alt2.noarch.rpm
perl-UNIVERSAL-require-0.11-alt1.noarch.rpm

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

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

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

Re: [devel] perl noarch on x86_64

[Mikhail Gusarov]

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

Twas brillig at 15:10:56 20.11.2008 UTC+03 when mithraen@altlinux.ru did gyre and gimble:

Первый же попавшийся пакет

 ДС> perl-Data-ICal-0.13-alt1.noarch.rpm

содержит в i386-linux только пустую директорию /usr/lib/perl5/vendor_perl/i386-linux/auto/Data/ICal

Впрочем, неаккуратненько.

-- 

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

Re: [devel] perl noarch on x86_64

[Dmitry V. Levin]

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

On Thu, Nov 20, 2008 at 06:16:56PM +0600, Mikhail Gusarov wrote:
> Twas brillig at 15:10:56 20.11.2008 UTC+03 when mithraen@altlinux.ru did gyre and gimble:
> 
> Первый же попавшийся пакет
> 
>  ДС> perl-Data-ICal-0.13-alt1.noarch.rpm
> 
> содержит в i386-linux только пустую директорию /usr/lib/perl5/vendor_perl/i386-linux/auto/Data/ICal
> 
> Впрочем, неаккуратненько.

И второй попавшийся, с пустым
/usr/lib/perl5/vendor_perl/i386-linux/auto/GnuPG/Interface

Может быть, в brp-cleanup чего-то надо добавить?


-- 
ldv

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

Re: [devel] perl noarch on x86_64

[Nikolay A. Fetisov]

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

On Thu, 20 Nov 2008 15:10:56 +0300
Денис Смирнов wrote:

> Эти пакеты noarch, но при этом содержат что-либо внутри /usr/lib/perl5/vendor_perl/i386-linux
> мне почему-то кажется что на самом деле их нельзя считать noarch

Они просто нарушают Perl Packaging Policy.

Поскольку сделавший их cpan2rpm положил в пакет целиком
%perl_vendor_privlib, а не %perl_vendor_privlib/<Module_name>* 
В итоге, на какой архитектуре noarch-пакет собрался, такой 
%perl_vendor_privlib/%arch-linux/auto/<Module_name>  в нём и будет.


И, по-моему, список не полный, ещё быть noarch-пакеты с 
/usr/lib/perl5/vendor_perl/x86_64-linux/auto/*

-- 
С уважением,
Николай Фетисов

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

Re: [devel] perl noarch on x86_64

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

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

On Thu, Nov 20, 2008 at 03:47:31PM +0300, Nikolay A. Fetisov wrote:

>> Эти пакеты noarch, но при этом содержат что-либо внутри /usr/lib/perl5/vendor_perl/i386-linux
>> мне почему-то кажется что на самом деле их нельзя считать noarch
NAF> Они просто нарушают Perl Packaging Policy.

В любом случае:
- наличие _файлов_ внутри таких подкаталогов noarch пакетов -- это
  однозначно blocker, ибо работать такой пакет не будет, и это надо
  проверять sisyphus_check'ом;
- наличие пустых _каталогов_ -- проверять repocop'ом, и, видимо, генерить
  патчи;

Кстати о. А не мог бы repocop генерить патчи заодно и чтобы вообще не
класть в пакет целиком %perl_vendor_privlib? А то по этой причине
некоторые perl пакеты сейчас не собираются, а фиксить их ручками так не
хочется... :)

NAF> Поскольку сделавший их cpan2rpm положил в пакет целиком
NAF> %perl_vendor_privlib, а не %perl_vendor_privlib/<Module_name>* 
NAF> В итоге, на какой архитектуре noarch-пакет собрался, такой 
NAF> %perl_vendor_privlib/%arch-linux/auto/<Module_name>  в нём и будет.
NAF> И, по-моему, список не полный, ещё быть noarch-пакеты с 
NAF> /usr/lib/perl5/vendor_perl/x86_64-linux/auto/*

Именно так, спасибо. Вот список провинившихся:

perl-SNMP_Session-1.08-alt1.noarch.rpm
perl-Test-Deep-0.093-alt1.noarch.rpm
perl-Test-NoWarnings-0.082-alt1.noarch.rpm
perl-Test-Tester-0.103-alt1.noarch.rpm

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

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

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

Re: [devel] perl noarch on x86_64

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

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

On Thu, Nov 20, 2008 at 03:18:42PM +0300, Dmitry V. Levin wrote:

DVL> И второй попавшийся, с пустым
DVL> /usr/lib/perl5/vendor_perl/i386-linux/auto/GnuPG/Interface
DVL> Может быть, в brp-cleanup чего-то надо добавить?

Кстати мне понадобилось один раз запаковать .gitignore. Можно какой-то
ручкой отучить brp-cleanup удалять его? :)

А в brp-cleanup возможно:
if [ -d $RPM_BUILD_ROOT/usr/lib/perl5/vendor_perl ]; then
	find $RPM_BUILD_ROOT/usr/lib/perl5/vendor_perl -type d -empty -delete
fi

будет приемлимо, или это уже слишком?

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

http://freesource.info


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

Re: [devel] perl noarch on x86_64

[Igor Vlasenko]

On Thu, Nov 20, 2008 at 05:02:35PM +0300, Денис Смирнов wrote:
> В любом случае:
> - наличие _файлов_ внутри таких подкаталогов noarch пакетов -- это
>   однозначно blocker, ибо работать такой пакет не будет, и это надо
>   проверять sisyphus_check'ом;
> - наличие пустых _каталогов_ -- проверять repocop'ом, и, видимо, генерить
>   патчи;
> Кстати о. А не мог бы repocop генерить патчи заодно и чтобы вообще не
> класть в пакет целиком %perl_vendor_privlib? А то по этой причине
> некоторые perl пакеты сейчас не собираются, а фиксить их ручками так не
> хочется... :)

Я уже генерировал патчи (правда, не repocop'ом), в рассылку писал 
и даже сделал NMU Мише Бочкареву на 76 пакетов.

можно и репокопом.
логика там такая:
if (truly arch code, but noarch) {
    fail && patch {
-BuildArch: noarch
} 
if (truly arch code, %perl_vendor_archlib packaged) {
	warn && patch {
s/%perl_vendor_privlib/%perl_vendor_archlib/ in %files
}
if (noarch, but empty dirs and other garbage in archlib) {
    warn && patch {
+%exclude %perl_vendor_archlib
}

-- 

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


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

Re: [devel] perl noarch on x86_64

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

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

On Thu, Nov 20, 2008 at 04:36:23PM +0200, Igor Vlasenko wrote:

IV> Я уже генерировал патчи (правда, не repocop'ом), в рассылку писал 
IV> и даже сделал NMU Мише Бочкареву на 76 пакетов.

А скриптиком поделиться?

IV> +%exclude %perl_vendor_archlib

Это работает, но не самое лучшее решение :) repocop в отличии от
анализаторов spec'а знает что именно собирется в бинарник. И поэтому может
даже просто перечислить файлики.

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

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

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

Re: [devel] perl noarch on x86_64

[Igor Vlasenko]

On Thu, Nov 20, 2008 at 05:59:33PM +0300, Денис Смирнов wrote:
> On Thu, Nov 20, 2008 at 04:36:23PM +0200, Igor Vlasenko wrote:
> 
> IV> Я уже генерировал патчи (правда, не repocop'ом), в рассылку писал 
> IV> и даже сделал NMU Мише Бочкареву на 76 пакетов.
> 
> А скриптиком поделиться?

http://lists.altlinux.org/pipermail/devel/2008-October/161192.html

(предыстория см. в 
http://lists.altlinux.org/pipermail/devel/2008-October/161171.html
http://lists.altlinux.org/pipermail/devel/2008-October/161184.html
)

> IV> +%exclude %perl_vendor_archlib
> 
> Это работает, но не самое лучшее решение :) repocop в отличии от
> анализаторов spec'а знает что именно собирется в бинарник. И поэтому может
> даже просто перечислить файлики.

Список файлов зависит от версии, не хочется мусорить
в .spec мимолетными списками.

-- 

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


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

Re: [devel] perl noarch on x86_64

[Igor Vlasenko]

On Thu, Nov 20, 2008 at 05:02:35PM +0300, Денис Смирнов wrote:
> Именно так, спасибо. Вот список провинившихся:
> perl-SNMP_Session-1.08-alt1.noarch.rpm
> perl-Test-Deep-0.093-alt1.noarch.rpm
> perl-Test-NoWarnings-0.082-alt1.noarch.rpm
> perl-Test-Tester-0.103-alt1.noarch.rpm

Это еще тихие пакеты (пакуют пустые подкаталоги)
а вот 2 буйных, найденных запросом 
select distinct a.pkgid from rpm_files as a, rpm as b where a.pkgid=b.pkgid and filename glob '/usr/lib/perl5/*-linux/*' and arch='noarch'  and filemode & 16384=0; -- filemode is file

perl-Alien-wxWidgets-0.40-alt1.noarch
vicq-0.4.1-alt3.noarch

их надо лечить серьезно.
у первого похоже проблемы с Makefile.PL,
второй неправильно инсталлирует руками.

-- 

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


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

Re: [devel] perl noarch on x86_64

[Slava Dubrovskiy]

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

Igor Vlasenko пишет:
> Это еще тихие пакеты (пакуют пустые подкаталоги)
> а вот 2 буйных, найденных запросом 
> select distinct a.pkgid from rpm_files as a, rpm as b where a.pkgid=b.pkgid and filename glob '/usr/lib/perl5/*-linux/*' and arch='noarch'  and filemode & 16384=0; -- filemode is file
>
> perl-Alien-wxWidgets-0.40-alt1.noarch
> vicq-0.4.1-alt3.noarch
>
> их надо лечить серьезно.
> у первого похоже проблемы с Makefile.PL,
>   
Патчик дашь?  ;)
Или достаточно убрать noarch в спеке?

-- 
WBR,
Dubrovskiy Vyacheslav


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

Re: [devel] perl noarch on x86_64

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

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

On Thu, Nov 20, 2008 at 10:28:01PM +0200, Igor Vlasenko wrote:

IV> perl-Alien-wxWidgets-0.40-alt1.noarch
IV> vicq-0.4.1-alt3.noarch
IV> их надо лечить серьезно.
IV> у первого похоже проблемы с Makefile.PL,
IV> второй неправильно инсталлирует руками.

По vicq я и узнал об этой проблеме.
Пациент вылечен и улетел в incoming/

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

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

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

Re: [devel] perl noarch on x86_64

[Igor Vlasenko]

On Thu, Nov 20, 2008 at 10:49:38PM +0200, Slava Dubrovskiy wrote:
> Igor Vlasenko пишет:
>> Это еще тихие пакеты (пакуют пустые подкаталоги)
>> а вот 2 буйных, найденных запросом select distinct a.pkgid from 
>> rpm_files as a, rpm as b where a.pkgid=b.pkgid and filename glob 
>> '/usr/lib/perl5/*-linux/*' and arch='noarch'  and filemode & 16384=0; 
>> -- filemode is file
>>
>> perl-Alien-wxWidgets-0.40-alt1.noarch
>> vicq-0.4.1-alt3.noarch

> Или достаточно убрать noarch в спеке?
Да, достаточно убрать noarch в спеке.
хоть там и pm, но содержимое gtk2_2_8_8_uni_gcc_3_4.pm
arch-зависимо.

-- 

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


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

Re: [devel] perl noarch on x86_64

[Alexey Tourbin]

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

On Thu, Nov 20, 2008 at 03:18:42PM +0300, Dmitry V. Levin wrote:
> И второй попавшийся, с пустым
> /usr/lib/perl5/vendor_perl/i386-linux/auto/GnuPG/Interface

(Следует убедиться, что такие пакеты не проходят sisyphus_check.)

> Может быть, в brp-cleanup чего-то надо добавить?

Надо "разрезать" brp-cleanup наподобие find-requires.  Но это менее
актуальная задача.  В случае с перловыми пакетами желательна коррекция
на уровне spec-файлов, а не на уровне brp-cleanup.  В brp-cleanup
следовало бы удалять некоторые неиспользуемые каталоги через
'rmdir --ignore-fail-on-non-empty' (и тогда glob не будет их ловить).
Но это всё ещё допускает некоторый класс неправильной запаковки
преловых каталогов, причем эти случаи отловить автоматически будет
уже несколько сложнее (по сравнению с files.req).

Короче, мое мнение -- надо править spec-файлы.

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