compiled against kdbusaddons development package with related support
Details
Details
Diff Detail
Diff Detail
- Repository
- R297 KDED
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
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.