fvogt added a comment.
`${KDE_INSTALL_CONFDIR}` should be changed to somewhere else (but maybe in a
different patch)
IMO `file(APPEND ...)` is the wrong approach, it should collect all
categories in a variable and only write it out once at the end (with
configure_file). I'm not fluent in CMake, I'm not sure whether that's actually
possible that way.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D9446
To: mlaurent, kfunk, lbeltrame, cgiboudeaux, dfaure, fvogt
Cc: #frameworks, #build_system