Comment 1 for bug 153800

Revision history for this message
John S. Gruber (jsjgruber) wrote :

Attached is a patch to restricted-manager to put the options needed for Xgl and compiz in the screen section of xorg.conf so that displayconfig-gtk won't overwrite them. It seems to me that it is probably better for displayconfig-gtk to own the device section rather than have both it and restricted-manager try to manage the section's options. While displayconfig-gtk may add to the screen section, it seems to leave already existing options alone. In contrast, as mentioned, displayconfig-gtk creates a totally new device section when it makes a change. The nvidia-xconfig command circumvention above works for the same reason.

The allowglxwithcomposite option shouldn't hurt the newer nvidia versions (nvidia-glx and nvidia-glx-new) and the other options that are understood by the new versions are ignored by the old one (nvidia-glx-legacy). Therefore it should be safe to add them all at once in the nvidiadriver class rather than add individual enable_config_hook methods to the -new and -legacy driver classes. While the patch takes the simplification of simply altering the first screen section the existing code is already making the same choice.

Since the patch is for restricted-manager I took the liberty of adding that project to the bug report.