[devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Dmitry V. Levin]

[-- Attachment #1.1: Type: text/plain, Size: 524 bytes --]

Hi,

В новой сборке пакета texinfo (точнее говоря, info-install) реализован
файлтриггер, который теперь следит за тем, чтобы info index был всегда
в актуальном состоянии.
Мантейнерам спек-файлов больше не нужно следить за корректностью вызовов
%install_info, %uninstall_info и %__install_info.
Более того, теперь лучше все такие устаревшие вызовы из спек-файлов убрать.
Для напоминания я расставил вывод предупреждений как на стадии вычисления
устаревших макросов, так и в самих вызываемых утилитах.


-- 
ldv

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

[-- Attachment #2: Type: text/plain, Size: 168 bytes --]

_______________________________________________
devel-announce mailing list
devel-announce@lists.altlinux.org
https://lists.altlinux.org/mailman/listinfo/devel-announce

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Slava Semushin]

18 мая 2009 г. 3:15 пользователь Dmitry V. Levin <ldv@altlinux> написал:
[...]
> Для напоминания я расставил вывод предупреждений как на стадии вычисления
> устаревших макросов, так и в самих вызываемых утилитах.

Ещё бы репокопа этому обучить..

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 03:21:33AM +0700, Slava Semushin wrote:
> 18 мая 2009 г. 3:15 пользователь Dmitry V. Levin <ldv@altlinux> написал:
> [...]
> > Для напоминания я расставил вывод предупреждений как на стадии вычисления
> > устаревших макросов, так и в самих вызываемых утилитах.
> 
> Ещё бы репокопа этому обучить..
в процессе.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 03:21:33AM +0700, Slava Semushin wrote:
> 18 мая 2009 г. 3:15 пользователь Dmitry V. Levin <ldv@altlinux> написал:
> [...]
> > Для напоминания я расставил вывод предупреждений как на стадии вычисления
> > устаревших макросов, так и в самих вызываемых утилитах.
> 
> Ещё бы репокопа этому обучить..

Обучил, также добавил патч генератор 
(свежее письмо в [cyber]: repocop-patch +200 (1194))
патч генератор удаляет 
%install_info
%uninstall_info
%xemacs_install_info
%xemacs_uninstall_info
но пока не %__install_info (это тоже добавлю).

что касается %xemacs_install_info, то с одной стороны,
update-info-dir сейчас обрабатывает их некорректно,
но с другой стороны, как сказал Дима,

LDV> Все прямые и косвенные вызовы install-info 
LDV> из спек-файлов теперь игнорируются.

смысла в них теперь нет, а поправить update-info-dir 
дело 5 минут.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Slava Semushin]

18 мая 2009 г. 21:15 пользователь Igor Vlasenko
<vlasenko@imath.kiev.ua> написал:
>> > Для напоминания я расставил вывод предупреждений как на стадии вычисления
>> > устаревших макросов, так и в самих вызываемых утилитах.
>>
>> Ещё бы репокопа этому обучить..
>
> Обучил, также добавил патч генератор
> (свежее письмо в [cyber]: repocop-patch +200 (1194))
> патч генератор удаляет
> %install_info
> %uninstall_info
> %xemacs_install_info
> %xemacs_uninstall_info
> но пока не %__install_info (это тоже добавлю).

Он не удаляет

Requires(post): %install_info
Requires(preun): %uninstall_info

хотя  вроде бы должен.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Tue, May 19, 2009 at 05:07:01PM +0700, Slava Semushin wrote:
> Он не удаляет
> 
> Requires(post): %install_info
> Requires(preun): %uninstall_info
> 
> хотя  вроде бы должен.

Спасибо, исправил. Завтра будет обновление.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 12:15:56AM +0400, Dmitry V. Levin wrote:
> Hi,
> 
> В новой сборке пакета texinfo (точнее говоря, info-install) реализован
> файлтриггер, который теперь следит за тем, чтобы info index был всегда
> в актуальном состоянии.
> Мантейнерам спек-файлов больше не нужно следить за корректностью вызовов
> %install_info, %uninstall_info и %__install_info.
> Более того, теперь лучше все такие устаревшие вызовы из спек-файлов убрать.
> Для напоминания я расставил вывод предупреждений как на стадии вычисления
> устаревших макросов, так и в самих вызываемых утилитах.

Немного рано, есть еще тонкости, которые желательно
учесть в update-info-dir.

Напомню, свежие спеки, по которым можно пройтись grep, есть в
http://repocop.altlinux.org/pub/repocop/data/repocop-specfiles-current.tar.gz

В целом у нас установка более-менее стандартная,
за исключением xemacs-* пакетов (see xemacsen-0.4):

#------------------------------------------------------------------------------
# info related stuff
#------------------------------------------------------------------------------
%_xemacs_infodir %_infodir/xemacs
%_xemacs_infosection --section=XEmacs
%_xemacs_infodirfile --dir-file=%_xemacs_infodir/dir
%_xemacs_installinfo %__install_info %_xemacs_infosection %_xemacs_infodirfile

Сейчас update-info-dir вписывает в общий dir,
но правильно это делать в %_infodir/xemacs/dir

Так же по новой схеме важно, чтобы у info файлов были заполнены
INFO-DIR-SECTION и
START-INFO-DIR-ENTRY.

У нас нашелся всего один пакет без INFO-DIR-SECTION с явными 
--entry= --section=,
./ucblogo-6.0-alt1.src.spec:/sbin/install-info %_infodir/ucblogo.info --entry="* UCBLogo: (ucblogo).	Berkeley Logo User Manual." --section="Programming Languages"  %_infodir/dir 2>/dev/null || :

но думаю, что в Сизхифе должно водиться гораздо больше пакетов
и без INFO-DIR-SECTION, и без --entry= --section=.

Я могу добавить в repocop проверку info файлов 
на наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY,

но думаю, что для корректного переезда необходима
некоторая жестокость, и эту проверку на 
наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY
надо добавить прямо в sisyphus_check.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 12:45:56PM +0300, Igor Vlasenko wrote:
> Так же по новой схеме важно, чтобы у info файлов были заполнены
> INFO-DIR-SECTION и
> START-INFO-DIR-ENTRY.
> Я могу добавить в repocop проверку info файлов 
> на наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY,
> 
> но думаю, что для корректного переезда необходима
> некоторая жестокость, и эту проверку на 
> наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY
> надо добавить прямо в sisyphus_check.
s/sisyphus_check/rpm-build/.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 12:45:56PM +0300, Igor Vlasenko wrote:
> Так же по новой схеме важно, чтобы у info файлов были заполнены
> INFO-DIR-SECTION и
> START-INFO-DIR-ENTRY.
> 
> У нас нашелся всего один пакет без INFO-DIR-SECTION с явными 
> --entry= --section=,
> ./ucblogo-6.0-alt1.src.spec:/sbin/install-info %_infodir/ucblogo.info --entry="* UCBLogo: (ucblogo).	Berkeley Logo User Manual." --section="Programming Languages"  %_infodir/dir 2>/dev/null || :

Не совсем так, я пропустил многострочники,
основанные на %__install_info \ ...
Вот пример из automake_1.7.spec:
%post 
...
%__install_info \
        --section=Development/Other \
        --entry="* automake: (automake).                         Making Makefile
        --entry="* aclocal: (automake)Invoking aclocal.          Generating aclo

Правильно ли я понимаю, что теперь, чтобы 
избежать замусоривания %_infodir/dir, нам надо запретить
такие фокусы с --section= и  --entry=",
и вместо этого явно патчить .info файлы, тобы
корректные INFO-DIR-SECTION и START-INFO-DIR-ENTRY
они носили с собой внутри?

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Dmitry V. Levin]

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

On Mon, May 18, 2009 at 01:32:23PM +0300, Igor Vlasenko wrote:
> On Mon, May 18, 2009 at 12:45:56PM +0300, Igor Vlasenko wrote:
> > Так же по новой схеме важно, чтобы у info файлов были заполнены
> > INFO-DIR-SECTION и
> > START-INFO-DIR-ENTRY.
> > 
> > У нас нашелся всего один пакет без INFO-DIR-SECTION с явными 
> > --entry= --section=,
> > ./ucblogo-6.0-alt1.src.spec:/sbin/install-info %_infodir/ucblogo.info --entry="* UCBLogo: (ucblogo).	Berkeley Logo User Manual." --section="Programming Languages"  %_infodir/dir 2>/dev/null || :

Все прямые и косвенные вызовы install-info из спек-файлов теперь
игнорируются.

> Не совсем так, я пропустил многострочники,
> основанные на %__install_info \ ...
> Вот пример из automake_1.7.spec:
> %post 
> ...
> %__install_info \
>         --section=Development/Other \
>         --entry="* automake: (automake).                         Making Makefile
>         --entry="* aclocal: (automake)Invoking aclocal.          Generating aclo
> 
> Правильно ли я понимаю, что теперь, чтобы 
> избежать замусоривания %_infodir/dir, нам надо запретить
> такие фокусы с --section= и  --entry=",
> и вместо этого явно патчить .info файлы, тобы
> корректные INFO-DIR-SECTION и START-INFO-DIR-ENTRY
> они носили с собой внутри?

Да, только патчить надо texinfo-файлы.


-- 
ldv

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Igor Vlasenko]

On Mon, May 18, 2009 at 03:25:06PM +0400, Dmitry V. Levin wrote:
> Все прямые и косвенные вызовы install-info из спек-файлов теперь
> игнорируются.

Тогда надо подправить update-info-dir,
чтобы он был совместим с %_xemacs_installinfo.
Иначе сейчас это ломает справку xemacs.

> > корректные INFO-DIR-SECTION и START-INFO-DIR-ENTRY
> > они носили с собой внутри?
> 
> Да, только патчить надо texinfo-файлы.

Ок, я тогда постараюсь описать это на wiki.
И получается, по хорошему еще нужна проверка
brp-verify-info на наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY.

-- 

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Dmitry V. Levin]

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

On Mon, May 18, 2009 at 02:58:24PM +0300, Igor Vlasenko wrote:
> On Mon, May 18, 2009 at 03:25:06PM +0400, Dmitry V. Levin wrote:
> > Все прямые и косвенные вызовы install-info из спек-файлов теперь
> > игнорируются.
> 
> Тогда надо подправить update-info-dir,
> чтобы он был совместим с %_xemacs_installinfo.
> Иначе сейчас это ломает справку xemacs.

Да, конечно.

> > > корректные INFO-DIR-SECTION и START-INFO-DIR-ENTRY
> > > они носили с собой внутри?
> > 
> > Да, только патчить надо texinfo-файлы.
> 
> Ок, я тогда постараюсь описать это на wiki.
> И получается, по хорошему еще нужна проверка
> brp-verify-info на наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY.

Да, конечно.


-- 
ldv

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

Re: [devel] [devel-announce] I: obsolete %install_info/%uninstall_info/%__install_info macros

[Dmitry V. Levin]

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

On Mon, May 18, 2009 at 08:03:10PM +0400, Dmitry V. Levin wrote:
> On Mon, May 18, 2009 at 02:58:24PM +0300, Igor Vlasenko wrote:
> > On Mon, May 18, 2009 at 03:25:06PM +0400, Dmitry V. Levin wrote:
> > > Все прямые и косвенные вызовы install-info из спек-файлов теперь
> > > игнорируются.
> > 
> > Тогда надо подправить update-info-dir,
> > чтобы он был совместим с %_xemacs_installinfo.
> > Иначе сейчас это ломает справку xemacs.
> 
> Да, конечно.

texinfo-4.13-alt4 уже обучен обрабатывать каждый каталог в /usr/share/info
отдельно.

> > > > корректные INFO-DIR-SECTION и START-INFO-DIR-ENTRY
> > > > они носили с собой внутри?
> > > 
> > > Да, только патчить надо texinfo-файлы.
> > 
> > Ок, я тогда постараюсь описать это на wiki.
> > И получается, по хорошему еще нужна проверка
> > brp-verify-info на наличие INFO-DIR-SECTION и START-INFO-DIR-ENTRY.
> 
> Да, конечно.

rpm-build-4.0.4-alt98.10 уже проверяет info-файлы.
По умолчанию наличие неправильных info-файлов является ошибкой.
Изменить умолчание можно с помощью
%set_verify_info_method relaxed


-- 
ldv

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