Bug 4670 - не правильно обрабатываеться select для файла cplay_control
Summary: не правильно обрабатываеться select для файла cplay_control
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: cplay (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: partizan
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-04 08:10 MSD by Alexander Borovsky
Modified: 2006-12-17 19:08 MSK (History)
2 users (show)

See Also:

Attachments
Патч, исправляющий проблему (workaround) (294 bytes, patch)
2004-07-04 08:11 MSD, Alexander Borovsky 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Borovsky 2004-07-04 08:10:20 MSD 
При вызове select для данного файла всегда возвращаеться, что в нё есть данные
для чтения. Поэтому select сразу возвращает управления и нагрузка на процессор
состовляет 100%. Workaround: в FIFOControl.handlecommands добавить строку
time.sleep(0.02). Это увеличит время реакции, но нагрузка на процессор будет не
большой 
Steps to Reproduce:
1. создать /var/tmp/cplay_control
2. запустить cplay
Comment 1 Alexander Borovsky 2004-07-04 08:11:59 MSD 
Created attachment 472 [details]
Патч, исправляющий проблему (workaround)
Comment 2 Evgenii Terechkov 2006-02-14 19:36:34 MSK 
Ну не знаю, не знаю, я именно эту версию брал за основу (хакал под себя).
Никогда такого не видел, хотя пользуюсь постоянно (WM по горячим клавишам шлёт
строки в ~tmp/cplay_control). Ничего связанного с обработкой фифы (судя по
диффу) не тронул.

PS: посмотрел сайт этого финна и улыбнулся. Он почти в точности все те же фичи
(с)делал, которыми я больше года пользуюсь :-)
Comment 3 Evgenii Terechkov 2006-02-16 20:57:14 MSK 
Понял похоже откуда такая проблема! Неужели постящий багу создавал cplay_control
как обычный файл? Надо было как pipe.

Проверил: если вместо пайпа положить файл обычный, то проблема воспроизводится.
С пайпом проблем нет (бо мой хак специально создаёт в нужном месте пайп). Так
что должно лечиться или "mkfifo" в spec-е или патчем исходника, чтоб создавал
пайп где-нить в ~/tmp (как я и сделал).
Comment 4 Michael Shigorin 2006-09-27 17:23:06 MSD 
Так в команде с тех пор прибавление было, или уже не используете?  Для 1.49-alt4
эта бага актуальна или нет?
Comment 5 Evgenii Terechkov 2006-09-28 05:37:44 MSD 
(In reply to comment #4)
> Так в команде с тех пор прибавление было, или уже не используете?  Для 1.49-alt4
> эта бага актуальна или нет?

Если это мне вопрос: как наладил себе xmms - не использую. Про актуаольность -
нет вроде:

R -qp --changelog cplay-1.49-alt4.src.rpm H -16
* Пнд Мар 14 2005 Alexander Borovsky <partizan@altlinux.ru> 1.49-alt4

- Rebuild with new python

* Птн Авг 20 2004 Yuri N. Sedunov <aris@altlinux.ru> 1.49-alt3

- install systemwide config /etc/cplayrc.
- Fixed ogg-files navigation (Alexander Borovsky).
- playing mp3-files with .wav extension alllowed (Alexander Borovsky).
- install systemwide config /etc/cplayrc.

* Вск Июл 04 2004 Yuri N. Sedunov <aris@altlinux.ru> 1.49-alt2

- install /usr/bin/cplay as a symlink to /usr/lib/cplay/cplay.py
- fix #4670 (Alexander Borovsky).

Не смотрел только, как оно там пофикшено - предложенным патчем или кошерно как
то. А вооще-то cplay уже в orphaned, что должно о чём-то говорить.
Comment 6 Michael Shigorin 2006-12-17 19:08:02 MSK 
Ясно, значит, за orphaned/cplay-1.49-alt4.src.rpm просто забыли напоследок
закрыть баги.

Безглючная ему память.