Bug 6192

Summary: ThinLinc graphical installer fails to launch on Fedora 25 (wayland)
Product: ThinLinc Reporter: Henrik Andersson <hean01@cendio.se>
Component: Server InstallerAssignee: Henrik Andersson <hean01@cendio.se>
Status: NEW QA Contact: Bugzilla mail exporter <bugzilla-qa@cendio.se>
Severity: Normal    
Priority: P2 CC: aaron@cendio.se, ossman@cendio.se, samuel@cendio.se
Version: trunkKeywords: samuel_tester
Target Milestone: Next   
Hardware: PC   
OS: Unknown   
Acceptance Criteria:

Description From cendio 2017-03-14 16:40:44
When running the installer one is asked to enter password for sudo graphically.
After authentication no more graphical dialogs are shown. The tlinstaller
process is still running and a strace shows hang on 'read(0,'.

Both seen on Samuels and Henriks desktops and we are running wayland. No
XAUTHORITY in environment and after running 'xhost +' the problem is solved.
------- Comment #1 From cendio 2017-03-14 16:49:26 -------
This is what the terminal says:

> $ sudo ./install-server
> [sudo] password for samuel: 
> No protocol specified
> /usr/lib64/python2.7/site-packages/gtk-2.0/gtk/__init__.py:57: GtkWarning: could not open display
>   warnings.warn(str(e), _gtk.Warning)
> No protocol specified
> Could not start PyGTK. Using text mode fallback...
------- Comment #2 From cendio 2017-03-21 09:44:16 -------
https://bugzilla.redhat.com/show_bug.cgi?id=1274451 ..?
------- Comment #3 From cendio 2017-03-21 10:29:53 -------
Fun. I guess it is time for another rewrite of tl-setup then. :)
------- Comment #4 From cendio 2017-03-21 13:25:34 -------
(In reply to comment #2)
> https://bugzilla.redhat.com/show_bug.cgi?id=1274451 ..?

The short summary is that running graphical applications as root is now
forbidden. The suggested approach is some front-end/back-end solution, ideally
using PolicyKit.
------- Comment #5 From cendio 2017-03-21 13:37:34 -------
We'll do a platform specific note before release with a suggested workaround.

Then we'll move this bug to Next for finding an automatic workaround that
forces text mode.

After that we'll have to have a new discussion on where to go from there and
------- Comment #6 From cendio 2017-03-29 10:59:23 -------
Platform specific note added. Moving to Next.

(Note to self: fix list of bugs in release notes.)
------- Comment #7 From cendio 2017-04-06 14:45:01 -------
Platform Specific Note looks good, workaround works.