Review Request 119901: Fix ECM to use qmake instead of hardcoding plugin install dirs

Rohan Garg rohangarg at kubuntu.org
Mon Aug 25 14:09:23 UTC 2014



> On Aug. 23, 2014, 1:16 a.m., David Faure wrote:
> > Is this going to lead to "make install" installing plugins into $QTDIR rather than into $CMAKE_INSTALL_PREFIX? This is a big no-no by default. For a developer, the path returned by qmake might not even be writable (/usr/...).
> > 
> > QCA has an option to install into Qt, we could do the same here if you want, but it should be off by default. Distros can enable it when compiling packages for /usr.

Ah, yes, I'll fix it so that it doesn't break CMAKE_INSTALL_PREFIX.


- Rohan


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


On Aug. 22, 2014, 6:10 p.m., Rohan Garg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119901/
> -----------------------------------------------------------
> 
> (Updated Aug. 22, 2014, 6:10 p.m.)
> 
> 
> Review request for Build System and KDE Frameworks.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Use qmake to query dirs for plugins and imports instead of hardcoding them in ECM.
> 
> 
> Diffs
> -----
> 
>   kde-modules/KDEInstallDirs.cmake 880539b 
> 
> Diff: https://git.reviewboard.kde.org/r/119901/diff/
> 
> 
> Testing
> -------
> 
> Seems to work on my system.
> 
> 
> Thanks,
> 
> Rohan Garg
> 
>

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


More information about the Kde-frameworks-devel mailing list