[Merge] lp:~3v1n0/nux/x11-conffile-on-unity-only into lp:nux

Marco Trevisan (Treviño) mail at 3v1n0.net
Wed Jun 13 02:13:29 UTC 2018


Marco Trevisan (Treviño) has proposed merging lp:~3v1n0/nux/x11-conffile-on-unity-only into lp:nux.

Commit message:
X11-check-unity-support: only run when called from an unity session

The content of $XDG_CURRENT_DESKTOP is checked inside a sub-shell
not to affect the value of IFS for all the scripts sourced by the X session.

If running in Unity we should call unity_support_test, not inside gnome
ubuntu X sessions.

XDG_CURRENT_DESKTOP values are currently the followings:
 - unity before 17.10: Unity
 - unity after 17.10: Unity:Unity7:ubuntu
 - gnome session: GNOME
 - ubuntu gnome session: ubuntu:GNOME

Also, at debian level, don't consider the Xsession.d script as a config file anymore

This is just a script to be loaded, and not something the user is supposed to change,
so it should be removed on simple package removal.
To achieve this, as per maintainer-guide 5.3, using sym-links is the standard way.

Requested reviews:
  Unity Team (unity-team)
Related bugs:
  Bug #1768610 in nux (Ubuntu): "leftover conffile forces GNOME is software rendering"
  https://bugs.launchpad.net/ubuntu/+source/nux/+bug/1768610

For more details, see:
https://code.launchpad.net/~3v1n0/nux/x11-conffile-on-unity-only/+merge/347861
-- 
Your team Unity Team is requested to review the proposed merge of lp:~3v1n0/nux/x11-conffile-on-unity-only into lp:nux.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 2738 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20180613/2f3fb608/attachment.diff>


More information about the Ubuntu-reviews mailing list