Review Request 115683: Install app desktop files to share/applications, not in a kde5 subdir

Rex Dieter rdieter at gmail.com
Thu Feb 13 04:28:31 UTC 2014


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


fyi, http://standards.freedesktop.org/menu-spec/menu-spec-latest.html , supporting vendor prefixes are not only a good idea, but is strongly encouraged.  "...  it is recommended that providers of desktop-files ensure that all desktop-file ids start with a vendor prefix."

- Rex Dieter


On Feb. 12, 2014, 10:37 a.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115683/
> -----------------------------------------------------------
> 
> (Updated Feb. 12, 2014, 10:37 a.m.)
> 
> 
> Review request for Build System, Extra Cmake Modules, KDE Frameworks, and David Faure.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Install app desktop files to share/applications, not in a kde5 subdir
> 
> Given that binaries are all installed in PREFIX/bin, and have to avoid
> clashes, doing the same for desktop files is no great issue, and
> installing into a subdirectory of applications/ just complicates matters
> for client code that needs to refer to the desktop file (is it
> "kde5-foo[.desktop]", "kde5/foo[.desktop]" or just "foo[.desktop]"?).
> 
> 
> Diffs
> -----
> 
>   kde-modules/KDEInstallDirs.cmake 46e15c17d488d56f146aba0c2d420f74a22b9152 
> 
> Diff: https://git.reviewboard.kde.org/r/115683/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140213/93df9be6/attachment.html>


More information about the Kde-buildsystem mailing list