<div dir="ltr"><div>All of these arguments makes sense, so I guess I agree with reverting this change. I didn't know about the goal of reducing the number of environment variables. Also, the fact that wayland displays are per user makes it different from the X11 displays. Sorry for all the trouble!<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 18, 2015 at 12:34 AM, Sjoerd Simons <span dir="ltr"><<a href="mailto:sjoerd.simons@collabora.co.uk" target="_blank">sjoerd.simons@collabora.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, 2015-08-17 at 09:58 -0700, Bill Spitzak wrote:<br>
> On Mon, Aug 17, 2015 at 7:48 AM, Ray Strode <<a href="mailto:halfline@gmail.com">halfline@gmail.com</a>><br>
> wrote:<br>
><br>
> > Hi,<br>
> ><br>
> > > This reverts commit fb7e13021730d0a5516ecbd3712ea4235e05d24d.<br>
> ><br>
> > thanks, you've got my vote.<br>
> ><br>
> > Acked-by: Ray Strode <<a href="mailto:rstrode@redhat.com">rstrode@redhat.com</a>><br>
> ><br>
> > --Ray<br>
><br>
><br>
> Seems right to me question about the method of nesting Wayland in X<br>
> and X<br>
> in wayland;<br>
><br>
> > The original problem of running Weston in a window in an existing<br>
> > GNOME<br>
> > X11 session and getting applications unintentionally launched into<br>
> > Weston can be circumvented by letting Weston use a non-default<br>
> > socket<br>
> > name, leaving wayland-0 unused.<br>
><br>
> If Wayland is already running and using wayland-0 this would prevent<br>
> these<br>
> programs from using the X11 api. For instance if you wished to test<br>
> the X11<br>
> api inside a Wayland session.<br>
<br>
</span>Not really it just means you require a way to override the order in<br>
which they try their display backends (which e.g. for Gtk+ is<br>
possible). As Ray previously mentioned, keeping environment variables<br>
for corner cases is entirely fine.<br>
<span class=""><br>
> Would it make sense that if DISPLAY is set and WAYLAND_DISPLAY is not<br>
> set,<br>
> that a program capable of doing both should prefer the X11 api? This<br>
> would<br>
> mean that I could force use of the X11 api by unsetting<br>
> WAYLAND_DISPLAY.<br>
<br>
</span>That seems rather fragile (and really up to the toolkit/program to<br>
implement, wayland can't know if the program supports X11).<br>
<span class=""><br>
> This will require changes to the client (unless the wayland connect > was<br>
> changed to fail in this case, which I suspect is not a good idea)<br>
> which<br>
> probably explains why the idea of renaming the socket was suggested.<br>
><br>
> As for the patch, I agree. The Foundry software would check if<br>
> DISPLAY was<br>
> not set, and set it directly to ":0", so that X would work always (it<br>
> did<br>
> not open the display by name because it wanted to fix child programs<br>
> as<br>
> well). This was commercial software and this was demanded by qc. So<br>
> effectively they wanted X11 to work without an environment variable.<br>
<br>
</span>the problem with DISPLAY is that it's not namespaced per user, so if<br>
you have two users logged in on an X graphical session one will have<br>
 e.g. DISPLAY=:0 the other DISPLAY=:1. Because of that defaulting to :0<br>
on X isn't a great idea on multi-user systems.<br>
<br>
However for wayland, the sockets are namespaced per user as they reside<br>
in their respective XDG_RUNTIME_DIR, which means two people can start a<br>
graphical session and both use wayland-0 for their compositor just<br>
fine.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Sjoerd Simons <<a href="mailto:sjoerd.simons@collabora.co.uk">sjoerd.simons@collabora.co.uk</a>><br>
Collabora Ltd.<br>
<br>
</div></div></blockquote></div><br></div>