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

Bug #1606246 reported by Stephen M. Webb
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mir
Triaged
Low
Unassigned
appmenu-qt5
Fix Released
High
Marco Trevisan (Treviño)
appmenu-qt5 (Ubuntu)
Fix Released
High
Unassigned
Xenial
Fix Released
High
Unassigned
mir (Ubuntu)
Triaged
Low
Unassigned
unity8-desktop-session (Ubuntu)
Fix Released
High
Stephen M. Webb

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

Revision history for this message
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)
Revision history for this message
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
Revision history for this message
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
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

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

Revision history for this message
Mathew Hodson (mhodson) 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
Revision history for this message
Chris Halse Rogers (raof) wrote :

This would appear to have been fixed in Zesty with the 0.3.0+17.04.20161124.2-0ubuntu1 upload. I don't know why that didn't close this bug.

Changed in appmenu-qt5 (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package appmenu-qt5 - 0.3.0+16.04.20170216-0ubuntu1

---------------
appmenu-qt5 (0.3.0+16.04.20170216-0ubuntu1) xenial; urgency=medium

  * IconCache: get the proper theme path based on the fact we're using a
    themed icon or not (LP: #1600136)
  * IconCache: use $XDG_RUNTIME_DIR as preferred place where to save
    icons
  * AppMenuPlatformMenuBar: Don't initialize X11 related functions in
    other environments (LP: #1606246)

 -- Marco Trevisan (Treviño) <mail@3v1n0.net> Thu, 16 Feb 2017 22:57:56 +0000

Changed in appmenu-qt5 (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of the Stable Release Update for appmenu-qt5 has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Mathew Hodson (mhodson)
no longer affects: unity8-desktop-session (Ubuntu Xenial)
Revision history for this message
Michał Sawicz (saviq) wrote :

Syncing task from Mir.

Changed in mir (Ubuntu):
importance: Undecided → Low
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.