Bug#559516: FTBFS on various archs

[Free Ekanayaka]

Hi,

|--==> On Sat, 05 Dec 2009 06:37:51 +0100 (CET), Jaromír Mikeš <mira.mikes at seznam.cz> said:

  >>Od: Jaromír Mikeš <mira.mikes at seznam.cz>
  >>> Od: dann frazier <dannf at debian.org>
  >>
  >>> ----------------------------------------
  >>> Package: zita-convolver
  >>> Version: 2.0.0-1
  >>> Severity: serious
  >>> 
  >>> From
  >>>
  >>https://buildd.debian.org/fetch.cgi?&pkg=zita-convolver&ver=2.0.0-1&arch=hppa&stamp=1259953696&file=log:
  >>> [...]
  >>> touch debian/stamp-patched
  >>> dh_testdir
  >>> /usr/bin/make -C libs/
  >>> make[1]: Entering directory `/build/buildd/zita-convolver-2.0.0/libs'
  >>> g++ -g -O2  -Wall -I. -fPIC -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -O3 -mmmx
  >>> -msse -mfpmath=sse -ffast-math -funroll-loops -fpermissive  -c -o
  >>> zita-convolver.o zita-convolver.cc
  >>> cc1plus: error: unrecognized command line option "-mmmx"
  >>> cc1plus: error: unrecognized command line option "-msse"
  >>> cc1plus: error: unrecognized command line option "-mfpmath=sse"
  >>> make[1]: *** [zita-convolver.o] Error 1
  >>> make[1]: Leaving directory `/build/buildd/zita-convolver-2.0.0/libs'
  >>> make: *** [build-stamp] Error 2
  >>> dpkg-buildpackage: error: debian/rules build gave error exit status 2
  >>
  >>Hello,
  >>
  >>here are the lines from zita-convolver makefile:
  >>CPPFLAGS += -Wall -I. -fPIC -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -O3
  >>CPPFLAGS += -mmmx -msse -mfpmath=sse -ffast-math -funroll-loops -fpermissive
  >>
  >>I am going remove -mmmx -msse and -mfpmath=sse by patch system.
  >>Should be removed some others?

I feel that instead of removing these flags unconditionally from all
archs, we should have a whitelist of archs that support them, and have
these build flags enabled for them.

It's more work and I'm not completely sure what the performance benefit
would be, so you might first want to benchmark zita-convolver when built
with and without the optimisation flags.

Ciao!

Free