Avoids adding the ${KDevPlatform_SOURCE_DIR}/plugins dir to the
include dirs of everything linking the "normal" libs
Details
Details
- Reviewers
kfunk - Group Reviewers
KDevelop - Commits
- R32:22db7c2e9d3b: Add separate interface library KDev::IExecute for execute/iexecuteplugin.h
Diff Detail
Diff Detail
- Repository
- R32 KDevelop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Rest LGTM,
This is pretty similar to what we did to IMakeBuilder before.
kdevplatform/plugins/execute/CMakeLists.txt | ||
---|---|---|
32 | KDevPlatformShellExecute -> KDevPlatformExecute? |
Comment Actions
Ah, had not seen, okay, so following a pattern :) Makes me wonder if KDevPlatformExecute then should get some target export, as well right? And perhaps then be named KDevPlatformIExecute (_I_), to indicate the interface as well here?
kdevplatform/plugins/execute/CMakeLists.txt | ||
---|---|---|
32 | Sure, was a copy, paste & bad-adapt error, should be KDevPlatformExecute, fixed. |