Bug 16069

Summary: reverse multi-mode order
Product: Sisyphus Reporter: Michael Shigorin <mike>
Component: alterator-backend-x11Assignee: Michael Shigorin <mike>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: minor    
Priority: P2 CC: boyarsh, led, mike, sem, snejok, zerg
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
patch by led@ none

Description Michael Shigorin 2008-06-17 14:35:28 MSD
Created attachment 2675 [details]
patch by led@

xconf -R 1024x768,800x600 xorg.conf xorg.new приводит к тому, что режимы перечисляются в обратном порядке:

Modes    "800x600" "1024x768"

Это, возможно, где-то учитывается чётной багой, поскольку при автодетекте список идёт по убыванию.  Если так, не забыть проверить/поправить после исправления xconf.

Прилагаю патчик led@, который у меня приложился-собрался, но почему-то xconf -R 1024x768 old new привёл к появлению (оставлению?) в new полного списка режимов имени автодетекта, наблюдающегося в old.  В общем, надо посмотреть внимательней.
Comment 1 Michael Shigorin 2008-06-17 14:48:38 MSD
2 inger, slazav: это всё происходит у меня в ветке mike/M40.

PS насчёт порядка -- кажется, это tac в tools/resolution_autosetup::tail_res() 
Comment 2 Michael Shigorin 2009-10-30 13:38:46 MSK
Modes неактуальны.
Comment 3 Vladislav Zavjalov 2009-10-30 16:17:10 MSK
Я совсем недавно убирал этот tac... Проблема вылезла в связи с новым xconf, а  причем тут 40 - не знаю.

http://git.altlinux.org/people/slazav/packages/?p=alterator-x11.git;a=commit;h=ead7857a15070847da8b461e7cca94083d2aa448
соответственно, попало всюду, начиная с 1.0-alt1

Modes, по крайней мере в инсталяторе актуальны. А то как воспрянут сплющенные люди, а хозяева дистрибутивов к этому еще не готовы :)

А xconf я переписал в связи с отказом от libxorgconfig (http://lists.altlinux.org/pipermail/devel/2009-October/175831.html), так что разные такие странности сейчас возможны. Впрочем, мне все больше кажется, что стоит забрать libxorgconfig из xorg и восстановить старый xconf...
Comment 4 led 2009-10-30 18:08:56 MSK
(В ответ на комментарий №3)
> Впрочем, мне все больше кажется, что
> стоит забрать libxorgconfig из xorg и восстановить старый xconf...

+1
Тем более, что libxorgconfig в xorg уже очень давно заброшен и недоделан. Я ещё года два назад его исправлял, но мейнтейнер xorg отказался принимать исправления (по причине - "не хочу")
Comment 5 Vladislav Zavjalov 2009-11-02 04:48:10 MSK
Ошибка в патче видна в последней его строчке: нехорошо говорить "return AddListItem(ActionPtr, actions, action)", если у нас

 #define AddListItem(type, head, new) \
-    head = (type) xf86addListItem((GenericListPtr) (head), (GenericListPtr) (new));
+    (new)->list.next = NULL; \
+    MergeLists(type, head, new)


libxorgconfig забираю в alterator-x11. (Пока не ставлю библиотеку в систему, собираюсь с ней статически - правильно ли?)

Возвращаю старый xconf.

Патч прикладываю, так что теперь xconf не будет переворачивать списки видеорежимов (но ежели у кого был свой tac - у него уже поломалось и не починется).
Comment 6 Vladislav Zavjalov 2009-11-02 06:08:54 MSK
alterator-x11-1.4-alt1
Comment 7 Michael Shigorin 2009-11-02 16:26:38 MSK
(In reply to comment #5)
> Ошибка в патче видна в последней его строчке
Возможно, я уж не помню, почему сам его не смержил-пушнул.

> libxorgconfig забираю в alterator-x11. (Пока не ставлю библиотеку в систему,
> собираюсь с ней статически - правильно ли?)
Зачем?  Лучше забирай пакет.  Как минимум сейчас это даст время спокойно разобраться с форком части alterator-x11 по имени ltsp-x11-autosetup (который бы надо помержить, да никак не разберусь попробовать текущий alterator-x11-backend и выкатить список проблем/исправлений или решить, что подходит).

> Патч прикладываю, так что теперь xconf не будет переворачивать списки
> видеорежимов (но ежели у кого был свой tac - у него уже поломалось и не
> починется).
И то хорошо.  А у кого поломалось, ну ква.

Спасибо, Слав :)