Review Request 120016: Read mimeapps.list from XDG_CONFIG_HOME and XDG_CONFIG_DIRS

[Luc Menut]

> On août 31, 2014, 6:50 après-midi, David Faure wrote:
> > Good point, we don't implement that spec yet.  ($desktop-mimeapps.list is missing too).
> > 
> > I'm not sure the syntax QStringList = list1 << list2 is portable to all compilers. It reads to me like it would modify list1.... Isn't it enough to use "+" here?

I've just updated the patch tu use "+" isntead of "<<".

Is it needed to read $desktop-mimeapps.list for [Added Associations] and [Removed Associations]?
The patch proposed by Ryan Lortie on xdg at lists.freedesktop.org (Sun, Apr 13, 2014 - update to new mimeapps.list specification) adds:
"Note that the desktop-specific files can only be used for specifying the default application for a given type.  It is not possible to add or remove associations from these files."
and
"[Added Associations] and [Removed Associations] groups may only appear in the non-desktop-specific files (ie: in files that are actually named "mimeapps.list")."


- Luc


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120016/#review65610
-----------------------------------------------------------


On août 31, 2014, 11 matin, Luc Menut wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120016/
> -----------------------------------------------------------
> 
> (Updated août 31, 2014, 11 matin)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kservice
> 
> 
> Description
> -------
> 
> Read the mimeapps.list from XDG_CONFIG_HOME and XDG_CONFIG_DIRS too, according to mime-apps-spec 1.0 .
> http://standards.freedesktop.org/mime-apps-spec/mime-apps-spec-1.0.html#file
> 
> This improves interoperability with applications using current glib/gio which now saves user mimeapps.list in XDG_CONFIG_HOME.
> 
> regards,
> Luc Menut - Mageia
> 
> PS: I don't have write access to kde git, so could you commit the change if the patch looks fine. Thanks.
> 
> 
> Diffs
> -----
> 
>   src/kbuildsycoca/kmimeassociations.cpp efc16f4 
> 
> Diff: https://git.reviewboard.kde.org/r/120016/diff/
> 
> 
> Testing
> -------
> 
> tested with kservice 5.1.0 (Mageia Cauldron)
> 
> 
> Thanks,
> 
> Luc Menut
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140831/f86db35f/attachment.html>