Bug 29810

Summary: Ошибка отработки триггера
Product: Sisyphus Reporter: serpiph <serpiph>
Component: vlc-miniAssignee: 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)