Bug 22634 - broken re-reading of custom display resolution
Summary: broken re-reading of custom display resolution
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-ltsconf (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Michael Shigorin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 15333
  Show dependency tree
 
Reported: 2009-12-28 21:29 MSK by Michael Shigorin
Modified: 2010-01-06 01:02 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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