Hard-coded X11 calls cause Unity 8 to fail to run on desktop

Bug #1606246 reported by Stephen M. Webb on 2016-07-25
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mir
Low
Unassigned
appmenu-qt5
High
Marco Trevisan (Treviño)
appmenu-qt5 (Ubuntu)
High
Unassigned
Xenial
High
Unassigned
unity8-desktop-session (Ubuntu)
High
Stephen M. Webb
Xenial
High
Unassigned

Bug Description

The appmenu-qt5 code has hardcoded Xlib calls that cause Unity 8 (or any other Qt-based applicaiton) to die on startup when run on Mir instead of X11.

This library extension should either not hardcode X11 (preferred) or check for the presence of a $DISPLAY environment variable and avoid code paths that assume X11.

The Unity 8 desktop currently has a hard-coded check for $QT_QPA_PLATFORMTHEME and unsets it as a workaround.

Related branches

lp:~bregma/unity8-desktop-session/lp-1590439
Christopher Townsend: Approve on 2016-07-28
lp:~3v1n0/appmenu-qt5/avoid-x11-calls-in-other-envs
Andrea Azzarone: Approve on 2016-12-05
lp:~3v1n0/appmenu-qt5/xenial-sru2
Approved for merging into lp:appmenu-qt5/16.04
Andrea Azzarone: Approve on 2017-02-13
Superseded for merging into lp:appmenu-qt5
Indicator Applet Developers: Pending requested 2017-02-13
Christopher Townsend (townsend) wrote :

The unity8-desktop-session task is for tracking the workaround until the proper fix can land in appmenu-qt5.

Changed in unity8-desktop-session (Ubuntu):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Stephen M. Webb (bregma)
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8-desktop-session - 1.0.13+16.10.20160726.1-0ubuntu1

---------------
unity8-desktop-session (1.0.13+16.10.20160726.1-0ubuntu1) yakkety; urgency=medium

  [ Chris Townsend ]
  * Reduce the priority of the Unity 8 session so it is will not appear
    as the default in Ubuntu 16.10. (LP: #1603102)

  [ Stephen M. Webb ]
  * load /etc/profile.d in Unity 8 desktop session (LP: #1590439,
    #1606246)
  * recommend additional packages to be co-installed with the Unity 8
    Desktop Session (LP: #1579551, #1591366, #1591394)

 -- <email address hidden> (Stephen M. Webb) Tue, 26 Jul 2016 13:32:01 +0000

Changed in unity8-desktop-session (Ubuntu):
status: In Progress → Fix Released
Changed in appmenu-qt5:
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
status: Triaged → In Progress
Daniel van Vugt (vanvugt) wrote :

Also affects Mir as mirrun contains the workaround of unsetting QT_QPA_PLATFORMTHEME. In future it would be nice if we didn't need that.

Changed in mir:
importance: Undecided → Low
status: New → Triaged
Changed in appmenu-qt5:
status: In Progress → Fix Released
Daniel van Vugt (vanvugt) wrote :

The task status still shows 'New' for 'appmenu-qt5 (Ubuntu)'. Is that right? No fix released yet?

Mathew Hodson (mathew-hodson) wrote :

Accepted appmenu-qt5 into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/appmenu-qt5/0.3.0+16.04.20170216-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in unity8-desktop-session (Ubuntu Xenial):
importance: Undecided → High
Changed in appmenu-qt5 (Ubuntu):
importance: Undecided → High
Changed in appmenu-qt5 (Ubuntu Xenial):
importance: Undecided → High
status: New → Fix Committed
tags: added: verification-needed
tags: added: verification-done
removed: verification-needed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers