Sounds good, will do! On Wed, Aug 12, 2015 at 4:41 AM, Pekka Paalanen <ppaala...@gmail.com> wrote:
> On Mon, 25 May 2015 01:12:15 -0700 > Dima Ryazanov <d...@gmail.com> wrote: > > > Although defaulting to wayland-0 seems convenient, it has an undesirable > > side effect: clients may unintentionally connect to the wrong compositor. > > Generally, it's safer to fail instead. Here's a real example: > > > > In Fedora 22, Gtk+ prefers Wayland over X11, though the default session > is still > > a normal X11 Gnome session. When you launch a Gtk+ app, it will try > Wayland, > > fail, then try X11, and succesfully start up. That works fine. > > > > Now suppose you launch Weston while running the Gnome session. Suddenly, > all > > of the Gtk+ apps launched from Gnome will show up inside Weston instead. > > That's unexpected. There's also no good way to prevent that from > happening > > (other than perhaps setting WAYLAND_DISPLAY to an invalid value when > launching > > an app). > > > > Not using wayland-0 as the default will solve that problem: an app > launched > > from the X11 Gnome session will use the X11 backend regardless of whether > > there's a wayland compositor running at the same time. > > > > Everything else should work as before. The compositor already sets > > the WAYLAND_DISPLAY when starting the session, so the lack of the > default value > > should not make a difference to the user. > > > > Signed-off-by: Dima Ryazanov <d...@gmail.com> > > Hi, > > gathering the comments from the thread, it seems we have Acked-bys from: > Pekka Paalanen <ppaala...@gmail.com> > Giulio Camuffo <giuliocamu...@gmail.com> > Daniel Stone <dan...@fooishbar.org> > "Jasper St. Pierre" <jstpie...@mecheye.net> > > Seems like a pretty strong set. Would you like to send a non-RFC > version of this patch? > > I think you can include also the above Acked-bys. > > > Thanks, > pq >
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel