Review Request 124009: tear cmake logic for qt4 and qt5 apart

Harald Sitter sitter at kde.org
Sun Jun 7 21:22:21 BST 2015


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

(Updated June 7, 2015, 8:22 p.m.)


Status
------

This change has been marked as submitted.


Review request for Build System, Phonon, Martin Gräßlin, and Hrvoje Senjan.


Changes
-------

Submitted with commit 6d7e80cd5b0361272bb801b213e4fabde9a01784 by Harald Sitter to branch master.


Repository: phonon


Description
-------

- qt4 as before
- qt5 now uses ECM to set compiler flags and so forth
- paths are still coming out of the joint finder file as the variable names
  in ECM are different and will potentially cause regressions


Diffs
-----

  cmake/CMakeLists.txt c31d4dd88d49a7aa2ed6df3c5349f34a5cdbf916 
  cmake/FindPhononInternal.cmake 045f777af4d4cb9c9cf707bf72102c92999f490b 
  cmake/PhononQt4.cmake PRE-CREATION 
  cmake/PhononQt5.cmake PRE-CREATION 

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


Testing
-------

ran cmake with both lineups and it passes, so yay, I guess.

kinda crap to test properly


Thanks,

Harald Sitter

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20150607/d8eaf2db/attachment.htm>
-------------- next part --------------
_______________________________________________
Kde-buildsystem mailing list
Kde-buildsystem at kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


More information about the kde-multimedia mailing list