[SCM] mpg123/master: Add -marm to CFLAGS to fix FTBFS on armhf.

Miguel Colon debian.micove at gmail.com
Thu Apr 5 22:32:36 UTC 2012


>    Add -marm to CFLAGS to fix FTBFS on armhf.
>
>    Closes: #667658
>    Thanks: peter green for the patch.
>

Hello:

I already had fixed this FTBFS by adding this:

ifeq ($(ARCH),armhf)
CONF_arm:=--with-cpu=neon
CFLAGS += -mfpu=neon
endif

This also added the optimizations that you wanted the package to build
with earlier but had FTBFS since Debian uses -mfpu=vfpv3-d16 by
default.

The patch was only needed if "--with-cpu=arm_nofpu" since the git
branch before the patch did not FTBFS. If the idea was to be able to
build with the current default "--with-cpu=neon" OR with a user
selected "--with-cpu=arm_nofpu" then it could have been added inside
the current IF as CFLAGS += -mfpu=neon -marm instead of adding another
IF check.

Cheers,
Miguel



More information about the pkg-multimedia-maintainers mailing list