Bug 409727 – media-video/vdr2jpeg-0.1.9 : fails to compile against libav-0.8.1

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Agostino Sarubbo 2012-03-26 08:16:41 UTC]

$summary

[Comment 1 Davide Pesavento (RETIRED) 2012-04-16 12:38:56 UTC]

build.log please :)

[Comment 2 Agostino Sarubbo 2012-04-16 17:46:38 UTC]

Created attachment 309159 [details]
build.log

Sorry, I don't know why there isn't a build log attached

[Comment 3 Davide Pesavento (RETIRED) 2012-04-17 15:00:10 UTC]

Created attachment 309265 [details, diff]
vdr2jpeg-0.1.9-libav.patch

Try this patch please.

[Comment 4 Alexis Ballier 2012-04-17 15:45:44 UTC]

(In reply to comment #3)
> Created attachment 309265 [details, diff] [details, diff]
> vdr2jpeg-0.1.9-libav.patch
> 
> Try this patch please.

@@ -245,7 +246,6 @@
                 big_picture.quality = (int)ist->st->quality;
             }else
                 big_picture.quality = (int)ost->st->quality;
-                big_picture.pict_type = AV_PICTURE_TYPE_NONE;
 //            big_picture.pts = AV_NOPTS_VALUE;
             big_picture.pts= ost->sync_opts;
 //            big_picture.pts= av_rescale(ost->sync_opts, AV_TIME_BASE*(int64_t)enc->time_base.num, enc->time_base.den);


you can probably remove that hunk:

@@ -245,7 +245,7 @@ static bool do_video_out(AVFormatContext
                 big_picture.quality = (int)ist->st->quality;
             }else
                 big_picture.quality = (int)ost->st->quality;
-                big_picture.pict_type = 0;
+                big_picture.pict_type = AV_PICTURE_TYPE_NONE;
 //            big_picture.pts = AV_NOPTS_VALUE;
             big_picture.pts= ost->sync_opts;
 //            big_picture.pts= av_rescale(ost->sync_opts, AV_TIME_BASE*(int64_t)enc->time_base.num, enc->time_base.den);


in the ffmpeg patch i committed some time ago, if this works for you.

[Comment 5 Tomáš Chvátal (RETIRED) 2012-04-21 08:01:28 UTC]

I updated the ffmpeg patch with two missing things for libav.
The mathematics.h is not included in avutil so i had to include that too.
The AV_PICTURE_TYPE_NONE is not defined in libav so I replaced it with properly typed variable so it work for both projects.

[Comment 6 Davide Pesavento (RETIRED) 2012-04-21 10:40:44 UTC]

(In reply to comment #5)
> I updated the ffmpeg patch with two missing things for libav.
> The mathematics.h is not included in avutil so i had to include that too.
> The AV_PICTURE_TYPE_NONE is not defined in libav so I replaced it with
> properly typed variable so it work for both projects.

Thanks!
I had removed the line with AV_PICTURE_TYPE_NONE altogether because an upstream developer (BBB on #libav) explained that that initialization was not needed. Anyway it works fine with 0 too, as long as it compiles :)

[Comment 7 Joerg Bornkessel (RETIRED) 2012-04-22 13:13:14 UTC]

Thanks for help out, ...