Review Request 126618: Make CMake macro kcoreaddons_desktop_to_json() work on windows.

Kåre Särs kare.sars at iki.fi
Sun Jan 3 13:22:37 UTC 2016


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

Review request for KDE Frameworks.


Repository: kcoreaddons


Description
-------

Without this patch kcoreaddons_desktop_to_json() will not find the destop file.

On windows GenericDataLocation returns "C:/Users/<USER>/AppData/Local" or "C:/ProgramData". That is not a path that contains any destip files ;)

This patch adds AppLocalDataLocation to the seach if the previous search does not return a match. 

Another option would be to hardcode the absolute path to all places that uses kcoreaddons_desktop_to_json(), but that does not feel too nice...

What other options do we have?


Diffs
-----

  src/lib/plugin/desktopfileparser.cpp 1122af8 

Diff: https://git.reviewboard.kde.org/r/126618/diff/


Testing
-------

KTextEditor compiles again on windows


Thanks,

Kåre Särs

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


More information about the Kde-frameworks-devel mailing list