[devel] IA: intltool-0.37.1-alt1 broken deps

[devel] IA: intltool-0.37.1-alt1 broken deps

[Dmitry V. Levin]

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

Hi,

По результатам ещё не закончившейся тестовой пересборки складывается
ощущение, что у пакета intltool-0.37.1-alt1 не хватает зависимостей.

В логах сборки нескольких десятков пакетов, которые перестали собираться
совсем недавно, присутствует следующая строка:

checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool

По крайней мере, "require XML::Parser" присутствует в файлах
/usr/bin/intltool-extract
/usr/bin/intltool-merge
/usr/share/aclocal/intltool.m4
(правда, в защищённом от perl.req виде).


-- 
ldv

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Alexey Tourbin]

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

On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> По крайней мере, "require XML::Parser" присутствует в файлах
> /usr/bin/intltool-extract
> /usr/bin/intltool-merge
> /usr/share/aclocal/intltool.m4
> (правда, в защищённом от perl.req виде).

/usr/bin/intltool-extract:
   595  sub readXml
   596  {
   597      my $xmldoc = shift || return;
   598      my $ret = eval 'require XML::Parser';
   599      if(!$ret) {
   600          die "You must have XML::Parser installed to run $0\n\n";
   601      }

Зачищать зависимости таким способом, как правило, не следует.
Стандартное сообщение об ошибке

$ perl -e 'use asdf'
Can't locate asdf.pm in @INC (@INC contains: /etc/perl5 ...) at -e line 1.
BEGIN failed--compilation aborted at -e line 1.
$

не намного хуже, чем user-friendly "You must have XML::Parser ...".
К тому же 'require XML::Parser' может отвалить по разным причинам,
а не только по той единственной причине, что XML::Parser не установлен.

Здесь, правда, есть тонкости (но они не касаются >~90% случаев).
Например, если в перловом исходнике стоит use (без eval/строки --
eval "use ..." это другое), то syntax check будет невозможен,
если соответствующий модуль не установлен (даже eval { use ... }).
"use" безусловно выполняется так скоро, как его видит парсер.
Чтобы сделать syntax check возможным без установки модуля, можно
заменить "use" на "require".  Но "use" может опережающим образом менять
прототипы функций, то есть, фактически, ->import может воздействовать
на синтаксис последующего кода.

Короче, в общем, этот код лучше развернуть следующим образом:
вместо
   598      my $ret = eval 'require XML::Parser';
   599      if(!$ret) {
   600          die "You must have XML::Parser installed to run $0\n\n";
   601      }
написать
	require XML::Parser;

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Alexey Tourbin]

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

On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> 
> По крайней мере, "require XML::Parser" присутствует в файлах
> /usr/bin/intltool-extract
> /usr/bin/intltool-merge
> /usr/share/aclocal/intltool.m4
> (правда, в защищённом от perl.req виде).

С другой стороны, следовало бы разобраться, насколько XML::Parser
там реально нужен.

$ rpmpeek /ALT/Sisyphus/files/noarch/RPMS/intltool-0.37.1-alt1.noarch.rpm grep -rn MSG . |grep XML
./usr/share/aclocal/intltool.m4:117:   AC_MSG_CHECKING([for XML::Parser])
./usr/share/aclocal/intltool.m4:121:       AC_MSG_ERROR([XML::Parser perl module is required for intltool])
$

Быть может, если здесь изничтожить суперобязательное требование
на XML::Parser, то часть пакетов "починится".  А может быть и нет.

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Dmitry V. Levin]

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

On Sun, Mar 09, 2008 at 09:00:30PM +0300, Alexey Tourbin wrote:
> On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> > checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> > 
> > По крайней мере, "require XML::Parser" присутствует в файлах
> > /usr/bin/intltool-extract
> > /usr/bin/intltool-merge
> > /usr/share/aclocal/intltool.m4
> > (правда, в защищённом от perl.req виде).
> 
> С другой стороны, следовало бы разобраться, насколько XML::Parser
> там реально нужен.

Цитирую intltool/README:

Details of the AC_PROG_INTLTOOL macro
-------------------------------------------

The first parameter indicates the minimum required version. The
configure script will halt if the version is older than the first
parameter.

The second parameter is to tell intltool that we don't need the
extended xml parsing abilities provided by the XML::Parser perl
module. If it is not provided, or is any value other than "no-xml",
then XML::Parser will be checked for by the configure script. This
feature is only available in intltool 0.31 or newer.


-- 
ldv

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Alexey Rusakov]

On Mon, 10 Mar 2008 22:19:56 +0300
Dmitry V. Levin wrote:

> On Sun, Mar 09, 2008 at 09:00:30PM +0300, Alexey Tourbin wrote:
> > On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> > > checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> > > 
> > > По крайней мере, "require XML::Parser" присутствует в файлах
> > > /usr/bin/intltool-extract
> > > /usr/bin/intltool-merge
> > > /usr/share/aclocal/intltool.m4
> > > (правда, в защищённом от perl.req виде).
> > 
> > С другой стороны, следовало бы разобраться, насколько XML::Parser
> > там реально нужен.
> 
> Цитирую intltool/README:
> 
> Details of the AC_PROG_INTLTOOL macro
> -------------------------------------------
> 
> The first parameter indicates the minimum required version. The
> configure script will halt if the version is older than the first
> parameter.
> 
> The second parameter is to tell intltool that we don't need the
> extended xml parsing abilities provided by the XML::Parser perl
> module. If it is not provided, or is any value other than "no-xml",
> then XML::Parser will be checked for by the configure script. This
> feature is only available in intltool 0.31 or newer.
То есть нужен всё-таки в зависимости от собираемого пакета.

В общем, в качестве совсем быстрого исправления я всё же отправил intltool
с явно проставленным Requires: perl-XML-Parser. А дальше нужно решать,
фиксить ли 69 пакетов на предмет BuildRequires или ставить патч в
intltool, приводящий к жёсткой зависимости на perl-XML-Parser. Если
тупо стремиться к минимально необходимому числу зависимостей, то стоит идти
первым путём. Но лично меня вполне устроит второй.

-- 
  Alexey "Ktirf" Rusakov
  GNOME Project
  ALT Linux Team

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Dmitry V. Levin]

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

On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> По результатам ещё не закончившейся тестовой пересборки складывается
> ощущение, что у пакета intltool-0.37.1-alt1 не хватает зависимостей.
> 
> В логах сборки нескольких десятков пакетов, которые перестали собираться
> совсем недавно, присутствует следующая строка:
> 
> checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool

По этой причине сломалась сборка 69 пакетов.

> По крайней мере, "require XML::Parser" присутствует в файлах
> /usr/bin/intltool-extract
> /usr/bin/intltool-merge
> /usr/share/aclocal/intltool.m4
> (правда, в защищённом от perl.req виде).


-- 
ldv

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Dmitry V. Levin]

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

On Mon, Mar 10, 2008 at 07:36:59PM +0300, Dmitry V. Levin wrote:
> On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> > По результатам ещё не закончившейся тестовой пересборки складывается
> > ощущение, что у пакета intltool-0.37.1-alt1 не хватает зависимостей.
> > 
> > В логах сборки нескольких десятков пакетов, которые перестали собираться
> > совсем недавно, присутствует следующая строка:
> > 
> > checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> 
> По этой причине сломалась сборка 69 пакетов.

Это я к тому, что проблема срочная и серьёзная, поэтому если мантейнер
не готов исправить её прямо сейчас, просьба сообщить об этом.


-- 
ldv

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

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Alexey Rusakov]

On Mon, 10 Mar 2008 22:14:59 +0300
Dmitry V. Levin wrote:

> On Mon, Mar 10, 2008 at 07:36:59PM +0300, Dmitry V. Levin wrote:
> > On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> > > По результатам ещё не закончившейся тестовой пересборки складывается
> > > ощущение, что у пакета intltool-0.37.1-alt1 не хватает зависимостей.
> > > 
> > > В логах сборки нескольких десятков пакетов, которые перестали собираться
> > > совсем недавно, присутствует следующая строка:
> > > 
> > > checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> > 
> > По этой причине сломалась сборка 69 пакетов.
> 
> Это я к тому, что проблема срочная и серьёзная, поэтому если мантейнер
> не готов исправить её прямо сейчас, просьба сообщить об этом.
В процессе, сегодня ночью будет пакет.

-- 
  Alexey "Ktirf" Rusakov
  Head of Systems development dept.
  ALT Linux Technology

Re: [devel] IA: intltool-0.37.1-alt1 broken deps

[Dmitry V. Levin]

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

On Tue, Mar 11, 2008 at 12:47:29AM +0300, Alexey Rusakov wrote:
> On Mon, 10 Mar 2008 22:14:59 +0300, Dmitry V. Levin wrote:
> > On Mon, Mar 10, 2008 at 07:36:59PM +0300, Dmitry V. Levin wrote:
> > > On Sun, Mar 09, 2008 at 08:00:43PM +0300, Dmitry V. Levin wrote:
> > > > По результатам ещё не закончившейся тестовой пересборки складывается
> > > > ощущение, что у пакета intltool-0.37.1-alt1 не хватает зависимостей.
> > > > 
> > > > В логах сборки нескольких десятков пакетов, которые перестали собираться
> > > > совсем недавно, присутствует следующая строка:
> > > > 
> > > > checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool
> > > 
> > > По этой причине сломалась сборка 69 пакетов.
> > 
> > Это я к тому, что проблема срочная и серьёзная, поэтому если мантейнер
> > не готов исправить её прямо сейчас, просьба сообщить об этом.
> В процессе, сегодня ночью будет пакет.

Спасибо.


-- 
ldv

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