[PATCH 180/180] Look for $PREFIX/etc/X11/xorg.conf before /etc/X11/xorg.conf
oreaus at gmail.com
Mon Aug 27 17:15:05 PDT 2012
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
> 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...
More information about the wayland-devel