Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 246148 - media-sound/picard-0.10-r1 cannot use media-video/ffmpeg-0.4.9_p20081014
Summary: media-sound/picard-0.10-r1 cannot use media-video/ffmpeg-0.4.9_p20081014
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Santiago M. Mola (RETIRED)
URL: http://musicbrainz.org/doc/PicardQt
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-09 12:09 UTC by Markus Rathgeb
Modified: 2009-05-02 13:05 UTC (History)
6 users (show)

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


Attachments
media-sound/picard-0.10-r2.ebuild (picard-0.10-r2.ebuild,2.35 KB, text/plain)
2008-11-09 12:11 UTC, Markus Rathgeb
Details
picard-0.10.0_ffmpeg_decode_audio2_support.patch (picard-0.10.0_ffmpeg_decode_audio2_support.patch,579 bytes, patch)
2008-11-09 12:16 UTC, Markus Rathgeb
Details | Diff
picard-0.10-avcodev-version.patch (picard-0.10-avcodev-version.patch,714 bytes, patch)
2008-11-29 11:07 UTC, Lars Strojny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Rathgeb 2008-11-09 12:09:38 UTC
Because in media-video/ffmpeg-0.4.9_p20081014 the deprecated function "avcodec_decode_audio" was removed the sources of picard have to be adapted to use "avcodec_decode_audio2".

I hope, that the changes I made are right (it is working for me).

I reported the problem upstream.
http://forums.musicbrainz.org/viewtopic.php?id=1369

Reproducible: Always

Steps to Reproduce:
Comment 1 Markus Rathgeb 2008-11-09 12:11:31 UTC
Created attachment 171160 [details]
media-sound/picard-0.10-r2.ebuild

The updated ebuild that use the patch (if a special ffmpeg version is installed)
Comment 2 Markus Rathgeb 2008-11-09 12:16:02 UTC
Created attachment 171162 [details, diff]
picard-0.10.0_ffmpeg_decode_audio2_support.patch

The patch to use not the deprecated function.
Comment 3 Wormo (RETIRED) gentoo-dev 2008-11-10 01:48:28 UTC
Thanks for submitting your patch to support newer ffmpeg, assigning to maintainers.
Comment 4 Lars Strojny 2008-11-29 11:07:41 UTC
Created attachment 173733 [details, diff]
picard-0.10-avcodev-version.patch

Use the pre processor so that we don't have to patch conditionally
Comment 5 Alexis Ballier gentoo-dev 2008-11-29 12:35:56 UTC
avcodec_decode_audio was such:
int avcodec_decode_audio(AVCodecContext *avctx, int16_t *samples,
                         int *frame_size_ptr,
                         const uint8_t *buf, int buf_size){
    *frame_size_ptr= AVCODEC_MAX_AUDIO_FRAME_SIZE;
    return avcodec_decode_audio2(avctx, samples, frame_size_ptr, buf, buf_size);
}


so in the patch the equivalent would be to set output_size to AVCODEC_MAX_AUDIO_FRAME_SIZE before calling avcodec_decode_audio2; moreover you can do this unconditionally as this function has been there and like that for years.
Comment 6 Markus Rathgeb 2008-11-29 13:19:29 UTC
@Lars Strojny
I believe that is better, too. Thanks
Comment 7 Santiago M. Mola (RETIRED) gentoo-dev 2008-12-01 21:19:22 UTC
Patch applied (as applied upstream) in 0.10-r2.

Thanks to all of you.
Comment 8 Gokdeniz Karadag 2009-05-02 13:05:44 UTC
(In reply to comment #7)
> Patch applied (as applied upstream) in 0.10-r2.
> 

Hi, 0.10-r2 is not in the tree. It is not present in CVS either
http://sources.gentoo.org/viewcvs.py/gentoo-x86/media-sound/picard/

But probably stabilizing 0.11 is a better idea ?