Bug 29810

Summary:	Ошибка отработки триггера
Product:	Sisyphus	Reporter:	serpiph <serpiph>
Component:	vlc-mini	Assignee:	Anton Farygin <rider>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	blocker
Priority:	P3	CC:	aen, boyarsh, darktemplar, rider, sbolshakov
Version:	unstable
Hardware:	all
OS:	Linux
Bug Depends on:
Bug Blocks:	27685

Description serpiph 2014-02-06 11:05:54 MSK

В триггере для vlc-mini написана команда

grep -qs '^/usr/lib/vlc/plugins/.*\.so$'

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

Для обхода приходится давать команду

apt-get --reinstall install vlc-mini

после каждого процесса обновления.

Comment 1 Evgenii Terechkov 2014-02-06 12:34:12 MSK

Насколько я помню, наоборот, при установке/обновлении на stdint filetrgger-а подаётся список файлов пакета.

Я запускал для тестирования:
===================================================================
#rpm -ql vlc-plugin-ffmpeg| sh -ex /usr/lib/rpm/vlc.filetrigger
+ grep -qs '^/usr/lib64/vlc/plugins/.*\.so$'
+ /usr/lib64/vlc/vlc-cache-gen /usr/lib64/vlc/plugins
===================================================================

но оно отрабатывает нормально.

Comment 2 serpiph 2014-02-06 12:39:15 MSK

При установке/обновлении да, подаётся. А если пакет стоит и не обновляется, то данных, похоже, нет. Иначе почему вторая команда (apt-get ...) отрабатывает как надо, а при установке/обновлении других программ от триггера идёт ошибка.

Comment 3 serpiph 2014-02-06 12:42:09 MSK

Похоже, фишка в том, что если grep не находит в обновлённых данных плагинов vlc, то выдает код выхода 1, который в bash превращается в ошибку всего файлтриггера, так как это последняя команда триггера.

Comment 4 AEN 2014-02-06 14:02:48 MSK

В p7 тоже, поспешили положить.
Прошу исправить asap, обновление сломано, -- блокер.

Comment 5 serpiph 2014-02-06 14:35:28 MSK

Вот правильный пример (из xscreensaver):

#!/bin/sh -e

grep -qs '^/etc/X11/xscreensaver/hack.d/' && update-xscreensaver ||:

Обратите внимание на завершение строки.

Comment 6 Anton V. Boyarshinov 2014-02-06 14:39:57 MSK

vlc.git=2.1.3-alt0.M70P.2

Comment 7 Anton V. Boyarshinov 2014-02-06 14:40:46 MSK

Точнее говоря, исправлено в p7,  а в сизифе надо исправить, поторопился закрывать

Comment 8 Repository Robot 2014-02-06 16:19:31 MSK

vlc-2.1.3-alt2 -> sisyphus:

* Thu Feb 06 2014 Sergey Bolshakov <sbolshakov@altlinux> 2.1.3-alt2
- filetrigger fixed (closes: #29810)