apol added a comment.

  Not saying that this patch is wrong, would have to look into it more closely.
  
  But for knsrc files you want to use KDE_INSTALL_KNSRCDIR anyway, which will 
fall into /usr.

INLINE COMMENTS

> KDEInstallDirs.cmake:376
> +        list(FIND _gnu_install_special_vars "${varname}" _vars_list_offset)
> +        if ((NOT _prefix_list_offset EQUAL -1 OR CMAKE_INSTALL_PREFIX 
> MATCHES "^/opt/") AND NOT _vars_list_offset EQUAL -1)
> +            set(KDE_INSTALL_${varname} "${CMAKE_INSTALL_FULL_${varname}}"

Why's /opt special?

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D26245

To: pwojcik, kossebau, alexmerry
Cc: apol, kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, 
michaelh, ngraham, bruns

Reply via email to