Bug 16335 - broken script reorganization
Summary: broken script reorganization
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-backend-x11 (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Mikhail Efremov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 16425
  Show dependency tree
 
Reported: 2008-07-10 20:42 MSD by Michael Shigorin
Modified: 2008-07-24 13:01 MSD (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2008-07-10 20:42:49 MSD
# config-x11
[...]
/usr/bin/x11_autosetup: line 116: setup_xorg_serial: command not found
mouseconf: You should specify Identifier and Driver
Setting up serial device /dev/ttyS1 as mouse
/usr/bin/x11_autosetup: line 116: setup_xorg_serial: command not found
mouseconf: You should specify Identifier and Driver
Setting up serial device /dev/ttyS0 as mouse

Результирующий xorg.conf взлетел на автоконфигурации мыши (imps2).

PS 2 slazav: если честно, так пока и не понял, какой был смысл перелопачивать *_autosetup и что от этого стало лучше.  Пару мест, которые сломались, и одно, которое стало хуже (одна из функций теперь в двух скриптах одинаковая, причём и там, и там её надо будет фиксить вместе с xconf) -- уже наблюдал.  Ещё мержить изменения по M40 от этого никак не легче.
Comment 1 Vladislav Zavjalov 2008-07-11 11:45:59 MSD
(In reply to comment #0)
> # config-x11
> [...]
> /usr/bin/x11_autosetup: line 116: setup_xorg_serial: command not found

Ошибка. Исправлю сейчас.

> PS 2 slazav: если честно, так пока и не понял, какой был смысл перелопачивать
> *_autosetup и что от этого стало лучше.  Пару мест, которые сломались, и одно,
> которое стало хуже (одна из функций теперь в двух скриптах одинаковая,
> причём и там, и там её надо будет фиксить вместе с xconf) -- уже наблюдал.  Ещё
> мержить изменения по M40 от этого никак не легче.

Смысл, был в попытке разобраться и упорядочить autosetup, который по-разному происходил в installer'е, backend'е, и x11_autosetup. Мне сейчас кажется, что стало понятнее и проще. Запихать fallback на fbdev во все места того что было я не осилил, а сейчас это довольно естественно получилось...

Насчет склеивания *_autosetup в один скрипт x11_autosetup -- возможно, я погорячился и стоит все разрезать обратно... Аргументы были - что теперь x11_autosetup используется везде как единое целое, и что настройки разных частей могут быть связаны: разрешение экрана обычно связано с настройкой монитора, а в случае fbdev - с настройкой драйвера...

В 40 может быть получится иметь свой старый x11_autosetup (новый-то рассчитывает на libXiconfig), а все остальное - новое.
Comment 2 Vladislav Zavjalov 2008-07-11 12:08:45 MSD
fixed in 0.18-alt9
Comment 3 Michael Shigorin 2008-07-15 10:02:09 MSD
(In reply to comment #2)
> fixed in 0.18-alt9
Спасибо :)

(In reply to comment #1)
> Смысл, был в попытке разобраться и упорядочить autosetup
Обычно это лучше получается в левом бранче, который перед мержем в master проверяется боем :-)

> Запихать fallback на fbdev во все места того что было
А, вот как.  Этого ещё не заметил.

> Насчет склеивания *_autosetup в один скрипт x11_autosetup --
> возможно, я погорячился и стоит все разрезать обратно...
> Аргументы были - что теперь x11_autosetup используется везде
> как единое целое
Ну так единый фронтэнд и так можно было оставить :)

Скриптики просто получались элегантные и хорошо обозримые, 
а проблема с недостатком передаваемой информации IMHO вылазила
только в ddcreslist | ddcresbest (см. ниже).

> и что настройки разных частей могут быть связаны: разрешение экрана
> обычно связано с настройкой монитора, а в случае fbdev - с настройкой
> драйвера...
Эт да, но тут тоже сгоряча совсем не стоит.  И когда удаляешь функцию или
скрипт, помогает грепнуть на всякий дерево -- где ещё использовалось...

По поводу автонастройки разрешения проводил некоторое исследование весной,
вывод -- всё плохо.  Вкратце:

- для TFT надо ставить верхнее разрешение (родное),
  для CRT -- обычно второе сверху или ориентируясь на >=85Hz;
- нужный для определения бит более-менее определён в 2.0
  (это и rider@ говорил);
- ddcprobe умеет только версию 1.2 стандарта;
- есть набор из двух утилит -- кажется, edid_dump и edid_parse
  или что-то вроде того, который умеет 1.3 и автор грозился принять
  дампы 2.0 для обучения им разбиралки, _но_ это было в начале века
  и на почту с дампом он не отозвался;
- в убунте подобный костыль в каком-то виде наработали, но уже выкинули
  (IIRC вместе с Daniel Stone в качестве [невменяемого] майнтейнера);
- предполагается, что это дело X-сервера (точнее, конкретных модулей),
  поскольку так выходит одно место для написания необходимой поддержки,
  да и по особенностям чипов модулям видней.

Если интересно, можно утащить в отдельный баг, но пока не знаю,
что с этим делать.  Озадачил коллегу подумать. :)

> В 40 может быть получится иметь свой старый x11_autosetup
> (новый-то рассчитывает на libXiconfig), а все остальное - новое.
Для M40 я вроде уже сделал все правки, которые от alterator-backend-x11
тут требовались.  В смысле особо трогать не планирую.
Comment 4 Mikhail Gusarov 2008-07-15 11:14:22 MSD
> - в убунте подобный костыль в каком-то виде наработали, но уже выкинули

Правильно, в X-сервер и унесли

>   (IIRC вместе с Daniel Stone в качестве [невменяемого] майнтейнера);

Кому ещё быть более невменяемым, чем апстримному разработчику :)

P.S.: что-то я никого, активно коммитящего в X.org-овый код, в альтовой team не наблюдаю.