Bug 5090

Summary: Linux client installation modifies defaults.list, which contains configuration about default applications
Product: ThinLinc Reporter: Henrik Andersson <hean01>
Component: Client platformsAssignee: Peter Åstrand <astrand>
Status: CLOSED FIXED    
Severity: Normal CC: samuel
Priority: P2 Keywords: prosaic, samuel_tester
Version: 4.1.1   
Target Milestone: 4.10.0   
Hardware: PC   
OS: Linux   
Acceptance Criteria:
* The Linux client package should not modify any "defaults.list".

Description Henrik Andersson cendio 2014-04-14 13:10:21 CEST 
Thinlinc clienconf mime type is still registered after an uninstall of the thinlinc client.

$ xdg-mime query default  application/vnd.cendio.thinlinc.clientconf
thinlinc-client.desktop
Comment 1 Peter Åstrand cendio 2014-04-14 13:32:44 CEST 
https://bugs.freedesktop.org/show_bug.cgi?id=71825
Comment 2 Peter Åstrand cendio 2018-12-05 09:32:14 CET 
Due to the work done on bug 7183, we are no longer using xdg-desktop-menu, and we are no longer updating /usr/local/share/applications/defaults.list.

As far as I understand it, it's not necessary to update this file when only one application provides a certain mime type. Some people also regard this file as obsolete, see: https://lists.freedesktop.org/archives/xdg/2014-February/013183.html

I've tested client uninstall (build 5976) at Ubuntu 18.10 and the problem is no longer present.
Comment 3 Peter Åstrand cendio 2018-12-05 15:49:08 CET 
Our conclusion is that it is actually incorrect for an application to touch defaults.list during installation; this file should be regarded as system administrator preferences.

Note that we will not try to undo changes done by old clients.
Comment 4 Samuel Mannehed cendio 2019-01-03 17:24:56 CET 
I tested the ThinLinc Client on Ubuntu 14.04, SLES 15 and Fedora 29. I verified the following:

* After installing the nightly ThinLinc Client it will properly be associated with .tlclient files in "Files" and nautilus

* Uninstalling the nightly ThinLinc Client removes this association

Seems to work well. Note that bug 7301 was found while testing this.