Bug 22634

Summary:	broken re-reading of custom display resolution
Product:	Sisyphus	Reporter:	Michael Shigorin <mike>
Component:	alterator-ltsconf	Assignee:	Michael Shigorin <mike>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	slazav
Version:	unstable
Hardware:	all
OS:	Linux
Bug Depends on:
Bug Blocks:	15333

Description Michael Shigorin 2009-12-28 21:29:30 MSK

Если в alterator-ltsconf выставить какому терминалу "своё" разрешение (например, 1024x768), то в /var/lib/ltsp/i586/etc/lts.conf оно в соответствующую секцию попадёт, но в UI модуля произойдёт сброс отображаемого значения на default.

Если после этого нажать "Defaults", результат соответствует ожидаемому (как в UI, так и в lts.conf).

Кажется, для проверки требуется развёрнутый экземпляр ALTSP... (утилита ltsconf вызывалась в чруте)

Comment 1 Vladislav Zavjalov 2009-12-28 22:09:40 MSK

Да, без возможности запуска я в api альтератора ничего не понимаю :(
Например, я не помню, не изменилось ли вот тут:
(let ((resolution (list-resolution text)))
?
Мне казалось, что text надо заменить на value, но я не вполне уверен...

Comment 2 Michael Shigorin 2010-01-05 12:49:26 MSK

Дело оказалось в этом коммите:
http://git.altlinux.org/people/slazav/packages/?p=alterator-ltsconf.git;a=commitdiff;h=93c5ec2184f8405edcbe3a92e5cc09feaa009f4d
-- разница получилась такова, что вместо условной обработки и безусловного вывода вышел условный вывод.

Ещё пришлось объехать двойное закавычивание -- хотя теперь код заодно стал более кавычкоустойчивым (для шелл-скрипта lts.conf годится и COMPCACHE=auto, и     X_MODE_0="1024x768"; а вот для альтератора следует квотить как положено).

http://git.altlinux.org/people/mike/packages/?p=alterator-ltsconf.git;a=commitdiff;h=303a27d9282a8a0c605b3a503b75687d11470aa7

В любом разе -- спасибо за обновление бэкенда и помощь, а это уже мелочи :-)

Comment 3 Repository Robot 2010-01-05 12:52:58 MSK

alterator-ltsconf-0.4.1-alt1 -> sisyphus:

* Tue Jan 05 2010 Michael Shigorin <mike@altlinux> 0.4.1-alt1

- backend fixup (closes: #22634)
- micro ui cleanup