Bug 15242 - libx86config - don't output "Disable" parameters from Section Module
Summary: libx86config - don't output "Disable" parameters from Section Module
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: xorg-x11-server (show other bugs)
Version: unstable
Hardware: all Linux
: P2 major
Assignee: Valery Inozemtsev
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-04 20:47 MSD by led
Modified: 2008-04-05 11:56 MSD (History)
1 user (show)

See Also:

Attachments
xorg-server-1.4.0.90-parser.patch (741 bytes, patch)
2008-04-04 20:49 MSD, led 		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 led 2008-04-04 20:47:56 MSD 
В библиотеке libx86config (которая у нас в пакете libxorg.config) не реализован 
вывод конструкций:
Disable "something"
Таким образом даже если они были в исходном файле, на выходе этих строк нет.

Патч следует.
Comment 1 led 2008-04-04 20:49:23 MSD 
Created attachment 2551 [details]
xorg-server-1.4.0.90-parser.patch

Собственно, патч. Очень прошу исправить:)
Comment 2 Valery Inozemtsev 2008-04-04 20:59:15 MSD 
в xorg.conf нет опции Disable
Comment 3 led 2008-04-04 21:02:34 MSD 
(In reply to comment #2)
> в xorg.conf нет опции Disable

xorg.conf(5):
......
MODULE SECTION
........
Disable  "modulename"
              This  instructs  the  server to not load the module called              
modulename.  Some modules are loaded by
              default in the server, and this overrides that default.
...........
Comment 4 Valery Inozemtsev 2008-04-04 21:09:41 MSD 
да, действительно...
но видимо у нас разные Module.c, ибо
                case DISABLE:
                        if (xf86getSubToken (&(ptr->mod_comment)) != STRING)
                                Error (QUOTE_MSG, "Disable");
                        ptr->mod_disable_lst =
                                xf86addNewLoadDirective (ptr->mod_disable_lst, 
val.str,
                                                                         
XF86_DISABLE_MODULE, NULL);
                        break;
Comment 5 led 2008-04-04 21:35:38 MSD 
(In reply to comment #4)
> да, действительно...
> но видимо у нас разные Module.c, ибо

Это не тот код, который выводит в результирующий файл. Посмотри патч, плиз.
Comment 6 Valery Inozemtsev 2008-04-04 21:42:01 MSD 
а, во, посмотрел. а шо патч так крив?
1.4.0.90-alt15
Comment 7 led 2008-04-04 21:59:27 MSD 
(In reply to comment #6)
> а, во, посмотрел. а шо патч так крив?

Ты про
#if 0
....
#endif
?
Потому что скопипастил, но "недочистил":)
Comment 8 Michael Shigorin 2008-04-05 11:56:20 MSD 
Ну и аюшки :)