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

[Rohan Garg]

> On Aug. 27, 2014, 11:35 p.m., Alex Merry wrote:
> > modules/ECMQueryQmake.cmake, line 2
> > <https://git.reviewboard.kde.org/r/119901/diff/6/?file=307834#file307834line2>
> >
> >     CMake macros/functions that take a variable name usually take it first. Swapping the argument order would make it more consistent with other things.

That's what I've done here? the variable being queried from qmake is passed as the first argument and the result is stored in the second argument?


- Rohan


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


On Aug. 27, 2014, 6:41 p.m., Rohan Garg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119901/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2014, 6:41 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
> -----
> 
>   modules/ECMQueryQmake.cmake PRE-CREATION 
>   kde-modules/KDEInstallDirs.cmake 880539b 
>   modules/ECMGeneratePriFile.cmake 34001d6 
> 
> 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-buildsystem/attachments/20140829/212d8b20/attachment.html>