Bug#533546: No zoran support in mplayer

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Jun 19 07:59:30 UTC 2009


On Fri, Jun 19, 2009 at 07:28:30AM +0200, Reinhard Tartler wrote:
> Reimar Döffinger <Reimar.Doeffinger at gmx.de> writes:
> > Simply forcing it via --enable-zr might work, but I recommend that
> 
> Not straight away, the configure script has an explicit check to fail if
> libavcodec_a is not built. I need to disable that check, similar to how
> Nikolay did in his latest followup.

IMO that is broken and also does not match the documented/expected
behaviour.
I think the test should be changed to something like this:
@@ -7274,13 +7305,13 @@
   #36067's seem to identify themselves as 36057PQC's, so the line
   #below should work for 36067's and 36057's.
   if grep -q -s -e "Multimedia video controller: Zoran Corporation ZR36057" /proc/pci ; then
-    _zr=yes
+    _zr=auto
   else
     _zr=no
   fi
 fi
-if test "$_zr" = yes ; then
-  if test "$_libavcodec_a" = yes ; then
+if test "$_zr" != no ; then
+  if test "$_libavcodec_a" = yes -o "$_zr" = yes ; then
     def_zr='#define CONFIG_ZR 1'
     _vomodules="zr zr2 $_vomodules"
   else


Though honestly I think the /proc/pci check is really bad anyway, maybe
Diego can think of something.





More information about the pkg-multimedia-maintainers mailing list