Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 138530 - xine-lib + dxr3 + non-mpeg12 stream: codec format not supported
Summary: xine-lib + dxr3 + non-mpeg12 stream: codec format not supported
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Diego Elio Pettenò (RETIRED)
URL: http://xinehq.de/
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-29 14:57 UTC by Alejandro Homs-Puron
Modified: 2006-06-29 17:22 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alejandro Homs-Puron 2006-06-29 14:57:05 UTC
Hi,

Maybe this is not the good place but ... I already submited a bug+fix to xine-lib developers @ sourceforce (#1424281), almost 5 months ago (2006-02-04). But no action has been taken. And each time I update my xine-lib ...

The problem arrives when reproducing non-mpeg1/2 streams through my RealMagic Hollywood+ (dxr3). I get the error:

[mpeg1video @ 0xb70c7d30]this codec supports only YUV420P

That was independenf of using internal or external
ffmpeg. After checking were the error came from
(libffmpeg/libavcodec/mpegvideo.c), and following a
little bit the code, I saw that the AVCodecContext
pixel format was not set in lavc_on_update_format
(xine_encoder.c) when it is allocated for encoding
MPEG1 for the dxr3. I then set it to PIX_FMT_YUVJ420P
and this solved the problem.

The patch is very simple:

==================================================================

--- xine-lib-1.1.1.orig/src/libffmpeg/xine_encoder.c    2005-05-06 06:22:51.000000000 +0200
+++ xine-lib-1.1.1/src/libffmpeg/xine_encoder.c 2006-02-04 22:36:16.000000000 +0100
@@ -158,6 +158,9 @@
     return 0;
   }

+  /* PIX_FMT */
+  this->context->pix_fmt = PIX_FMT_YUVJ420P;
+
   /* put sample parameters */
   this->context->bit_rate = drv->class->xine->config->register_range(drv->class->xine->config,
     "dxr3.encoding.lavc_bitrate", 10000, 1000, 20000,

==================================================================

Could you please include it in the new ebuilds? 

Thank you very much in advance

Alejandro
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-06-29 17:13:31 UTC
You'll probably have to wait for a bit before having it in portage (one day, maybe two, maybe three, depends on how much free time I can have in the next days). I'll apply it in upstream CVS and then refresh the snapshot as I was already planning.
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-06-29 17:22:09 UTC
Upstreamed, will be in next snapshot.
Thanks for reporting.