Bug 15242

Summary:

libx86config - don't output "Disable" parameters from Section Module

Product:

Sisyphus

Reporter:

led

Component:

xorg-x11-server

Assignee:

Valery Inozemtsev <shrek>

Status:

CLOSED FIXED

QA Contact:

qa-sisyphus

Severity:

major

Priority:

CC:

mike

Version:

unstable

Hardware:

all

OS:

Linux

Attachments:

Description	Flags
xorg-server-1.4.0.90-parser.patch	none

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

Ну и аюшки :)