[PATCH 180/180] Look for $PREFIX/etc/X11/xorg.conf before /etc/X11/xorg.conf

[Scott Moreau]

On Mon, Aug 27, 2012 at 6:05 PM, Bill Spitzak <spitzak at gmail.com> wrote:

> I realize now that this patch will only fix local (ie $PREFIX) installs of
> xwayland, and that something else needs to be done so that real system
> installs of wayland will work.
>
> The problem is if an /etc/X11/xorg.conf file exists, Xorg --wayland
> attempts to read it. If the text in the file causes the wayland backend to
> not work then it exits. This file may very well exist because the same
> machine is being used to run a legacy X server as well as wayland.
>
> I now think the best solution is to make Xorg --wayland use a different
> configuration file, perhaps /etc/X11/wayland.conf. This is especially true
> because I believe a non-existent config file will often work and this is
> the easiest way to indicate that when an xorg.conf file exists.
>

> Would love to have some feedback:
>
> 1. Is this really a bug or am I missing something obvious?
>
> 2. Is the idea of a different conf file the right one?


Yes, this does sound like a possible solution.


> Is my proposed name the right one?
>

I'd vote for xwayland.conf


>
> 3. If so is the best way to have weston add a --config switch when running
> Xorg, or modify the source code for Xorg?
>

Since there is a runtime check in for wayland in xserver already we can
probably just have it check for this file name and keep the path ordering
the same.


>
> 4. Is rearranging the search path (ie this patch) in Xorg to look in
> $PREFIX first a good idea anyway?


I don't think modifying the search paths would be necessary at all if the
configuration file name is unique.


> I and others on this list certainly expected it to work this way and it
> took some time before I figured out it was literally doing things
> differently than any other Linux software.
>

I'm not sure exactly what you mean by this. Software is software, it may
not always behave the way you expect.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20120827/21e86e66/attachment.html>