Review Request: Port Phonon five build system to ECM and Qt5Transitional.cmake

[Jon Severinsson]

> On Oct. 31, 2012, 4:58 p.m., David Faure wrote:
> > The circular dependency (due to libinqt5 being in kdelibs currently) will make building a bit harder though. Shouldn't ECM be fixed NOT to require libinqt5?

There is no circular dependency, as libinqt5 does not depend on Phonon. The repository layout may make it look like a circular dependency, but that is just an illusion. In reality, Phonon is just another tier 1 framework, and just like all other tier 1 frameworks it should depend on libinqt5.

To enable building all frameworks in one go, we could add phonon five as a git submodule in the frameworks branch, but personally I think that is overkill.


- Jon


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106970/#review21251
-----------------------------------------------------------


On Oct. 20, 2012, 6:56 p.m., Jon Severinsson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106970/
> -----------------------------------------------------------
> 
> (Updated Oct. 20, 2012, 6:56 p.m.)
> 
> 
> Review request for KDE Frameworks and Phonon.
> 
> 
> Description
> -------
> 
> This patch is based on my "Phonon five merge and cleanup" patch, and is a step towards building Phonon five on Qt5.
> 
> On Qt4 the build system will now insist on "libinqt5" being installed,
> which is located in the kdelibs repository in the frameworks branch.
> There is, however, no dependancy on the actual KDE Frameworks 5, and
> libinqt5 can be build and installed standalone.
> 
> 
> Diffs
> -----
> 
>   cmake/FindPhononInternal.cmake 20220a3 
>   demos/metadatareader/CMakeLists.txt fe6eb60 
>   demos/phota/CMakeLists.txt 2306c7e 
>   demos/simplecapture/CMakeLists.txt 4a2e1b0 
>   demos/simpleplayer/CMakeLists.txt 6824142 
> 
> Diff: http://git.reviewboard.kde.org/r/106970/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jon Severinsson
> 
>

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