Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 479394

Summary: media-video/cinelerra needs a patch for ffmpeg-2
Product: Gentoo Linux Reporter: Helmut Jarausch <jarausch>
Component: Current packagesAssignee: Gentoo Media-video project <media-video>
Status: RESOLVED FIXED    
Severity: normal Keywords: PATCH
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 476490    
Attachments: patch for ffmpeg-2

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!