Review Request 116098: Use KDEInstallDirs

Alexander Richardson arichardson.kde at googlemail.com
Sat Mar 1 11:14:19 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116098/
-----------------------------------------------------------

(Updated March 1, 2014, 11:14 a.m.)


Review request for KDE Frameworks, Daniel Nicoletti, Lamarque Souza, and Sebastian Kügler.


Changes
-------

Added libnm-qt hackers to the review request.


Repository: libnm-qt


Description
-------

When I build libnm-qt on my openSuSE 64bit system libnm-qt ends up installing into $prefix/lib instead of $prefix/lib64.
I know this can be fixed using -DLIB_SUFFIX=64, but KDEInstallDirs already handles this so why not use it.

E-C-M is already required, so that is no problem. Not sure however whether it is okay to use one of the KDE modules for libnm-qt.
If not I will update this request to use GNUInstallDirs which also handles the openSuSE case.

Not sure who is responsible for the qt5 branch, so I just added kdeframeworks as reviewers.


Diffs
-----

  CMakeLists.txt 9918278 
  NetworkManagerQt.pc.cmake 2c3ab07 

Diff: https://git.reviewboard.kde.org/r/116098/diff/


Testing
-------

Installed into the right directories after applying the patch.
grep -irn LIB_SUFFIX * returns nothing 


Thanks,

Alexander Richardson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140301/d6d34952/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list