Hi Reinhard,

On 02/27/2013 02:57 PM, Reinhard Tartler wrote:
> I don't understand this. Why does gcc on powerpcspe enable altivec by
> default? If powerpcspe ships a gcc that has broken default settings,
> then this should be fixed first.

I had a second look at this, and it turned out that it's actually a bug
in upstream's configure. Consider the attached patch: Some lines before,
configure decides, contrary to the default on generic powerpc, that
altivec is not available ("disable altivec", e.g. on powerpcspe).
However, in this case it still aborts and complains about some AltiVec
detail that should be ignored in case AltiVec is correctly detected as
disabled.

So using this new patch instead would be good.

An additional:

ifeq      ($(DEB_HOST_ARCH),powerpcspe)
confflags += --enable-pic -cpu=e500v2
endif

... in debian/confflags would be nice, though (similar to arch powerpc).

> I have noticed that you have filed a similar bug against mplayer, and
> suspect that you have filed many more bugs. Is it possible that with
> fixed gcc default settings, no changes would be necessary in mplayer,
> libav and possibly many other packages?

Right, I filed quite some bugs as you guessed. :-) Feel free to have a
look at

http://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=powerpcspe;[email protected]

Fortunately, libav's and mplayer2's configure are not very
representative. ;-) The majority of the other bugs are fixes in other
places or plain activation of powerpcspe.

Will file a similar upstream fix to the mplayer2 package/bug.

Roland
--- libav-0.8.5/configure.orig	2013-02-27 15:52:13.013650621 +0100
+++ libav-0.8.5/configure	2013-02-27 17:19:08.898946263 +0100
@@ -2730,10 +2730,12 @@
 EOF
 
         # check if our compiler supports braces for vector declarations
-        check_cc <<EOF || die "You need a compiler that supports {} in AltiVec vector declarations."
+        if enabled altivec ; then
+            check_cc <<EOF || die "You need a compiler that supports {} in AltiVec vector declarations."
 $inc_altivec_h
 int main (void) { (vector int) {1}; return 0; }
 EOF
+        fi
     fi
 
 elif enabled sparc; then
_______________________________________________
pkg-multimedia-maintainers mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to