<div dir="ltr">On Sun, Jan 27, 2013 at 11:03 PM, David Faure <span dir="ltr"><<a href="mailto:faure+bluesystems@kde.org" target="_blank">faure+bluesystems@kde.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sunday 27 January 2013 03:10:04 Harald Sitter wrote:<br>
</div><div class="im">> as discussed previously there is now a transitional library for qt5 called<br>
> phonon4qt5 and lives in a branch of the regular phonon git repos (pending<br>
> release).<br>
><br>
> phonon4qt5 is a 100% source compatible version of phonon4 that builds<br>
> against qt5. it is directly derived from master and as such should be kept<br>
> in sync whenever a change applies to both.<br>
><br>
> phonon4qt5 is meant to be as unintrusive as possible. in particular it has<br>
> a different soname (phonon4qt5) and include paths (prefix/phonon4qt5/) and<br>
> can be installed along side regular libphonon.<br>
><br>
> the cmake magic still talks about phonon, so you still<br>
> use phonon_INCLUDE_DIR and the likes to keep transitional changes to a<br>
> minimum. to that extent all that should be necessary for an application to<br>
> transit from libphonon to libphonon4qt5 is changing find_package from<br>
> phonon to phonon4qt5 (please note however that you must add the include_dir<br>
> to your includes as otherwise the headers may not be found - previously<br>
> with phonon often residing in /usr/include this would have worked even when<br>
> not explicitly adding it).<br>
<br>
</div>share/phonon4qt5/buildsystem/FindPhononInternal.cmake is included by<br>
Phonon4Qt5Config.cmake, and pollutes the application doing find_package quite a<br>
lot.<br>
It sets all sorts of install dirs, it (re)defines the "uninstall" target, and<br>
so on. All of this should be internal to phonon, not exposed to apps that use<br>
phonon.<br>
<br>
This leads to this problem in kdelibs-frameworks:<br>
When I apply this <a href="http://www.davidfaure.fr/2013/use_phonon4qt5.diff" target="_blank">http://www.davidfaure.fr/2013/use_phonon4qt5.diff</a><br>
I get this:<br>
<br>
-- Using CMake automoc builtin<br>
CMake Error at<br>
/d/kde/inst/kde_frameworks/share/phonon4qt5/buildsystem/FindPhononInternal.cmake:167<br>
(add_custom_target):<br>
  add_custom_target cannot create target "uninstall" because another target<br>
  with the same name already exists.  The existing target is a custom target<br>
  created in source directory "/d/kde/src/5/kdelibs-frameworks".  See<br>
  documentation for policy CMP0002 for more details.<br>
Call Stack (most recent call first):<br>
  /d/kde/inst/kde_frameworks/lib64/cmake/phonon4qt5/Phonon4Qt5Config.cmake:30<br>
(include)<br>
  CMakeLists.txt:416 (find_package)<br>
<br>
But the uninstall target isn't the only issue, all of FindPhononInternal.cmake<br>
is unwanted in apps. Why is it installed at all? For the phonon backends?<br></blockquote><div><br></div><div style>exactly</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


In that case the backends should include it explicitely, rather than<br>
Phonon4Qt5Config.cmake loading it.</blockquote><div><br></div><div style>agreed. probably was no problem up until now since all the sc4 stuff uses the finder over the cmake config, but the config indeed was incorrectly ported from the finder.</div>

<div style><br></div><div style>HS</div></div></div></div>