screen-share: Don't unset server environment variables
authorAndrew Wedgbury <andrew.wedgbury@realvnc.com>
Wed, 30 Apr 2014 08:51:19 +0000 (09:51 +0100)
committerKristian Høgsberg <krh@bitplanet.net>
Wed, 30 Apr 2014 23:54:31 +0000 (16:54 -0700)
commitbc51ad6afd5464d3b91b66e1c488a1fa7b4dc04c
tree8c03efa201169f56fc78af144cba15f365f55c6e
parent8c3c738718adf99d777cfdf37308806c231447ca
screen-share: Don't unset server environment variables

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 the RDP
backend, which overrides the automatic backend selection based on the
environment.

Keeping these environment variables allows the launched fullscreen shell server
to know the original server's display name, which it may need in order to show
a configuration UI.

Signed-off-by: Andrew Wedgbury <andrew.wedgbury@realvnc.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/screen-share.c