kossebau added a comment.

  (Just remember that using KDE_INSTALL_KNSRCDIR though needs at least 
KNewStuffCore from KF 5.57 (hint was missing in API dox, proposing D26248 
<https://phabricator.kde.org/D26248> to fix that).)
  
  Myself also need to reserve time to look closer at it, hoping to sneak in in 
the next days or week 1 of 2020
  
  Please see to add a unit test also covering this, so this special case is 
covered.

INLINE COMMENTS

> pwojcik wrote in KDEInstallDirs.cmake:376
> /usr is single special case, but any subdirectory of /opt/ is special by 
> GNUInstallDirs.cmake, following Filesystem Hierarchy Standard.

Ideally gets a small explaining comment in the code, as this is not directly 
obvious.

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