[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