[krita] 3rdparty: Comment out ffmpeg for now

[Dmitry Kazakov]

Hi, Boud!

As far as I remember, the initial idea was just to make Krita Installer 
automatically download and install FFMPEG from official site. I cannot 
remember, what was the problem with this approach?

On 30.05.2018 10:35, Boudewijn Rempt wrote:
> Git commit e2a5181d08e642a7a6328cb6215911905f979dda by Boudewijn Rempt.
> Committed on 30/05/2018 at 07:33.
> Pushed by rempt into branch 'master'.
>
> Comment out ffmpeg for now
>
> We would need to build our own ffmpeg, stripped from patent-encumbred
> codecs, which would confuse our users even more. Otherwise we are
> going to have real trouble, for instance with Krita in the Windows
> Store.
>
> https://www.ffmpeg.org/legal.html says
>
> Q: Is it perfectly alright to incorporate the whole FFmpeg core into my own commercial product?
>
> A: You might have a problem here. There have been cases where companies
> have used FFmpeg in their products. These companies found out that once
> you start trying to make money from patented technologies, the owners
> of the patents will come after their licensing fees. Notably, MPEG LA
> is vigilant and diligent about collecting for MPEG-related technologies.
>
> So we really need to consider this again :-(
>
> CCMAIL:kimageshop at kde.org
>
> M  +1    -1    3rdparty/CMakeLists.txt
>
> https://commits.kde.org/krita/e2a5181d08e642a7a6328cb6215911905f979dda
>
> diff --git a/3rdparty/CMakeLists.txt b/3rdparty/CMakeLists.txt
> index 987a66b89c2..568f8056280 100644
> --- a/3rdparty/CMakeLists.txt
> +++ b/3rdparty/CMakeLists.txt
> @@ -207,7 +207,7 @@ endif (ENABLE_PYTHON_DEPS OR NOT MINGW)
>   
>   if (MSVC OR MINGW)
>       add_subdirectory( ext_drmingw )
> -    add_subdirectory( ext_ffmpeg )
> +#    add_subdirectory( ext_ffmpeg )
>   endif (MSVC OR MINGW)
>   
>   if (NOT APPLE)

-- 
Dmitry Kazakov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kimageshop/attachments/20180601/fde3cf35/attachment-0001.html>