Re: ffmpeg patches

[Andreas Enge]

Hello,

On Wed, Feb 25, 2015 at 03:54:02PM +0100, Taylan Ulrich Bayırlı/Kammer wrote:
> - The second adds a (gnu packages audio) import to (gnu packages video),
>   which makes sense I think because video media usually contains audio,
>   but in my Audacity patch I had added (gnu packages video) to (gnu
>   packages audio) because Audacity uses FFmpeg, and that would be a
>   circular import.  I would say Audacity should go into its own module,
>   and we will have to remember that we cannot import video from audio.

that sounds convincing to me.

> Date: Mon, 23 Feb 2015 14:13:00 +0100
> Subject: [PATCH 1/3] gnu: ffmpeg: Fix comment.
> * gnu/packages/video.scm (ffmpeg): Remove libmp3lame from the comments listing
>   additional optional dependencies, because it's already been added on the
>   meanwhile.

Frankly, I would just include this silently into the second patch, dropping
a comment that has obviously become useless does not warrant its own commit
(others may have different opinons).

Is there a patch number 3?

Andreas