Review Request 111510: Consistently use FindFFMpeg.cmake from kdelibs

Alex Merry kde at randomguy3.me.uk
Mon Jul 15 22:48:26 UTC 2013



> On July 15, 2013, 5:12 p.m., Mark Kretschmann wrote:
> > Builds ok here with KDE 4.10.90. I'm wondering though if FindFFMpeg.cmake is also included in KDE 4.8, which we still support.

Just checked: yes, it is, and the docs at the top of the file suggest none of the variables have changed.


- Alex


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


On July 15, 2013, 1:18 p.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111510/
> -----------------------------------------------------------
> 
> (Updated July 15, 2013, 1:18 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> Consistently use FindFFMpeg.cmake from kdelibs
> 
> FindFFMpeg from kdelibs works differently to our internal one (and, in
> fact, more consistently with standard Find* files).  Since we depends on
> kdelibs anyway, we now just use the kdelibs one and don't bother
> shipping our own.
> 
> 
> Diffs
> -----
> 
>   ChangeLog 5fe5d2e64c771d722f3f90bf6c98d5013e56553c 
>   cmake/modules/FindFFmpeg.cmake 0aa2cdc33a88953fe8b94532cb45c43ad35e0da8 
>   src/CMakeLists.txt 589fddb9ea2177102cabd3a193a9e6848ea9c12d 
> 
> Diff: http://git.reviewboard.kde.org/r/111510/diff/
> 
> 
> Testing
> -------
> 
> Build works with FFMpeg and libOFA installed, and kdelibs 4.10.5.  No other configurations tested.  (Without the patch, the build fails with this configuration).
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130715/6d65396d/attachment-0001.html>


More information about the Amarok-devel mailing list