compiled against kdbusaddons development package with related support
Details
Details
Diff Detail
Diff Detail
- Repository
- R297 KDED
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
src/CMakeLists.txt | ||
---|---|---|
34 | Shouldn't that be ${KDE_INSTALL_FULL_BINDIR} instead? |
Comment Actions
I can update the patch with this path if you want. It came from the service template file
Exec=@CMAKE_INSTALL_PREFIX@/bin/kded5
Comment Actions
Indeed that was a bug in the .in files, arguably.
The executables get installed into KDE_INSTALL_BINDIR (says kde-modules/KDEInstallDirs.cmake), so if we want the full path version of it, it should be KDE_INSTALL_FULL_BINDIR.
src/CMakeLists.txt | ||
---|---|---|
34 | Remove the /bin, it's part of KDE_INSTALL_FULL_BINDIR... |
Comment Actions
Rechecked after applying D9480 (with a minor cmake 3.5 fix, see f316004e8a17) without any issue, please accept.