Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 479394 - media-video/cinelerra needs a patch for ffmpeg-2
Summary: media-video/cinelerra needs a patch for ffmpeg-2
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: ffmpeg-2.0
  Show dependency tree
 
Reported: 2013-08-01 12:09 UTC by Helmut Jarausch
Modified: 2013-08-07 02:35 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for ffmpeg-2 (cinelerra-ffmpeg2.patch,8.12 KB, patch)
2013-08-01 12:09 UTC, Helmut Jarausch
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Helmut Jarausch 2013-08-01 12:09:31 UTC
Created attachment 354814 [details, diff]
patch for ffmpeg-2

This patch should problaby be applied only if ffmpeg-2 is installed.
The patch is a bit ugly.
I've discovered too late that in (only) one place someone has introduced the
macro AVCODEC_MAX_AUDIO_FRAME_SIZE replacing the now missing MAX_AUDIO_FRAME_SIZE

I have defined MAX_AUDIO_FRAME_SIZE as recommended in several places.
Furthermore one has to replace CodecID  by AVCodecID
Comment 1 Alexis Ballier gentoo-dev 2013-08-02 12:56:21 UTC
Comment on attachment 354814 [details, diff]
patch for ffmpeg-2

thanks for the patch; there are series of patches improving ffmpeg support on the cinelerra ml but your patch is simpler for now.

>--- quicktime/ffmpeg/cmdutils.h.ORIG	2012-07-06 22:40:26.000000000 +0200
>+++ quicktime/ffmpeg/cmdutils.h	2013-08-01 13:14:20.840483614 +0200
>@@ -21,6 +21,7 @@
> 
> #ifndef FFMPEG_CMDUTILS_H
> #define FFMPEG_CMDUTILS_H
>+#define MAX_AUDIO_FRAME_SIZE 192000

for example here, this would not be needed if using avcodec_decode_audio4 but this is much more work

>+#define MAX_AUDIO_FRAME_SIZE
>+

why is it missing the value here ?
Comment 2 Alexis Ballier gentoo-dev 2013-08-07 02:35:10 UTC
cleaned up and applied, thanks!