Re: XDG_APPS_INSTALL_DIR and KStandardDirs

2013-12-28 Thread David Faure
On Tuesday 24 December 2013 15:46:34 Alex Merry wrote:
 This got a bit lost in the other thread on the kde4support tests...

Ah. I think I unconfused it a few minutes ago, didn't see this.

 I want to advocate setting XDG_APPS_INSTALL_DIR to be
 $CMAKE_INSTALL_PREFIX/share/applications instead of
 $CMAKE_INSTALL_PREFIX/share/applications/kde5.
 
 I think we should be putting our application desktop files directory in
 share/applications, not in share/applications/kde5.

I agree, but not for the reason below.
The reason below can be fixed by just fixing KStandardDirs.
But indeed we can make our lives easier by just using share/applications 
everywhere, like everyone else does.

David.

 The reason this has come up is that KStandardDirs currently contains a
 hack for how it compiles the value of xdgdata-apps.  The upshot of this
 is that if the directory $CMAKE_INSTALL_PREFIX/share/applications does
 not exist, KStandardDirs::resourceDirs(xdgdata-apps) does not contain
 that directory, but instead contains XDG_APPS_INSTALL_DIR, which is
 $CMAKE_INSTALL_PREFIX/share/applications/kde5.
 
 In normal circumstances this does not happen (because the directory
 normally does exist, and if it doesn't it's kind of irrelevant), but it
 does cause one of the unit test to fail on Jenkins.  Changing
 XDG_APPS_INSTALL_DIR would fix it, but it is not the only way to fix it.
  I just think that XDG_APPS_INSTALL_DIR *should* have a different value.
 
 Alex
 ___
 Kde-frameworks-devel mailing list
 Kde-frameworks-devel@kde.org
 https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

-- 
David Faure, fa...@kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


XDG_APPS_INSTALL_DIR and KStandardDirs

2013-12-24 Thread Alex Merry
This got a bit lost in the other thread on the kde4support tests...

I want to advocate setting XDG_APPS_INSTALL_DIR to be
$CMAKE_INSTALL_PREFIX/share/applications instead of
$CMAKE_INSTALL_PREFIX/share/applications/kde5.

I think we should be putting our application desktop files directory in
share/applications, not in share/applications/kde5.

The reason this has come up is that KStandardDirs currently contains a
hack for how it compiles the value of xdgdata-apps.  The upshot of this
is that if the directory $CMAKE_INSTALL_PREFIX/share/applications does
not exist, KStandardDirs::resourceDirs(xdgdata-apps) does not contain
that directory, but instead contains XDG_APPS_INSTALL_DIR, which is
$CMAKE_INSTALL_PREFIX/share/applications/kde5.

In normal circumstances this does not happen (because the directory
normally does exist, and if it doesn't it's kind of irrelevant), but it
does cause one of the unit test to fail on Jenkins.  Changing
XDG_APPS_INSTALL_DIR would fix it, but it is not the only way to fix it.
 I just think that XDG_APPS_INSTALL_DIR *should* have a different value.

Alex
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel