Re: [PATCH weston v2] screen-share: Don't unset server environment variables

2014-04-30 Thread Kristian Høgsberg
On Wed, Apr 30, 2014 at 10:53:30AM -0500, Jason Ekstrand wrote: > Looks good to me. > > Reviewed-by: Jason Ekstrand Thanks, committed. Kristian > On Apr 30, 2014 3:52 AM, "Andrew Wedgbury" > wrote: > > > There is no need to unset WAYLAND_DISPLAY and WAYLAND_SOCKET when > > screen-share > > l

Re: [PATCH weston v2] screen-share: Don't unset server environment variables

2014-04-30 Thread Jason Ekstrand
Looks good to me. Reviewed-by: Jason Ekstrand On Apr 30, 2014 3:52 AM, "Andrew Wedgbury" wrote: > There is no need to unset WAYLAND_DISPLAY and WAYLAND_SOCKET when > screen-share > launches the fullscreen shell server. This was done originally in case the > launched server decided to use the wa

[PATCH weston v2] screen-share: Don't unset server environment variables

2014-04-30 Thread Andrew Wedgbury
There is no need to unset WAYLAND_DISPLAY and WAYLAND_SOCKET when screen-share launches the fullscreen shell server. This was done originally in case the launched server decided to use the wayland backend based on the presence of these. However, we pass a command line argument telling it to use th