FS#47963 - [deepin-desktop-schemas] Don't override GNOME defaults

Attached to Project: Community Packages
Opened by Balló György (City-busz) - Sunday, 31 January 2016, 10:05 GMT
Last edited by Felix Yan (felixonmars) - Monday, 29 February 2016, 13:44 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Felix Yan (felixonmars)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Currently this package overrides some default settings for the GNOME session, which is not expected, I think.

See the attached diff file for PKGBUILD, which removes these override files, and leaves GNOME untouched.
   PKGBUILD.diff (0.5 KiB)
This task depends upon

Closed by  Felix Yan (felixonmars)
Monday, 29 February 2016, 13:44 GMT
Reason for closing:  Implemented
Additional comments about closing:  removed in 3.0.3-2. I've also requested upstream to remove it completely.
Comment by Balló György (City-busz) - Sunday, 31 January 2016, 14:58 GMT
However, Deepin synchronises the background with GNOME, which is bad, because Deepin unable to handle animated backgrounds including the GNOME default adwaita-timed.xml. I attached a diff file for the deepin-daemon package, which fixes the problem.
   deepin-daemon.diff (4 KiB)
Comment by Felix Yan (felixonmars) - Sunday, 31 January 2016, 16:17 GMT
I have asked upstream to remove those two files some time ago, and I will do that in next bump. However, I would suggest to submit the patch to upstream :)
Comment by Balló György (City-busz) - Sunday, 31 January 2016, 18:34 GMT
Patch submitted: https://github.com/linuxdeepin/dde-daemon/pull/7
Comment by Balló György (City-busz) - Monday, 01 February 2016, 07:08 GMT
Upstream wants to keep background sync with GNOME, so another patch was submitted and accepted, which fixes the problem too:
https://github.com/linuxdeepin/dde-daemon/commit/3cd63e46104116da4b67f806cacde1e9b894bdbf
Comment by Balló György (City-busz) - Friday, 26 February 2016, 18:02 GMT
The patch is included in deepin-daemon 3.0.11, so I think it's safe to remove these overrides.

Loading...