Review Request 128488: Add a fallback method for query_qmake() when there's no Qt5 installation

[David Faure]

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




modules/ECMQueryQmake.cmake (line 14)
<https://git.reviewboard.kde.org/r/128488/#comment65832>

    If query_qmake isn't public API, it's not what the cmakelists.txt is calling. So it looks like an internal error message surfacing up to the user.
    What should the user do when hitting this warning? (put qt5 qmake in PATH, right?). => I'd remove the mention of query_qmake() in the warning and explain more precisely what to do instead.
    
    And what should a developer do if they want to use ECM for qt4? (not use this file, I suppose, but if that file is used elsewhere in ECM, it's not really a choice, is it?)


- David Faure


On July 20, 2016, 8:42 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128488/
> -----------------------------------------------------------
> 
> (Updated July 20, 2016, 8:42 a.m.)
> 
> 
> Review request for Extra Cmake Modules, KDE Frameworks and David Faure.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Addresses its usage on systems where Qt5 isn't installed, it allows for modules using it to decide what they should do.
> 
> 
> Diffs
> -----
> 
>   modules/ECMQueryQmake.cmake 8f4cf17 
> 
> Diff: https://git.reviewboard.kde.org/r/128488/diff/
> 
> 
> Testing
> -------
> 
> Should fix this issue: https://build.kde.org/job/extra-cmake-modules%20master%20latest-qt4/16/PLATFORM=Linux,compiler=gcc/console
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20160721/11ab4892/attachment.html>