[Buildroot] mplayer-package doesn't build for other architectures than avr32

[Hans-Christian Egtvedt]

On Wed, 2007-08-29 at 10:33 +0200, Simon Pasch wrote:
> It's the same issues as in libmad-package:
> There are architecture-specific patches. (by the way...one should be
> removed, as they can't be used both at the same time.
> AVR32-Users...tell us which one is the good one!)

I did, but I think the email got drowned on the list.

Remove package/mplayer/mplayer-1.0rc1-avr32-support.patch

> I added an extra command to the Makefile, because the build fails on
> my x86. Don't know if this breaks the build on other architectures.
> So either a sed-command can be used inside the Makefile or an extra
> architecture-specific patch (but it's not that simple with $(ARCH),
> because then you need patches for i386, i686, etc).

-fomit-frame-pointer is not architecture dependant, so should be fine to
add it. I have not checked if it breaks the AVR32 build.

> This is my sed-command approach. (you have to manually rename the avr-patches):
> 
> --- buildroot/package/mplayer/mplayer.mk.old	2007-08-28 09:15:43.000000000 +0200
> +++ buildroot/package/mplayer/mplayer.mk	2007-08-29 10:02:07.000000000 +0200
> @@ -22,7 +22,9 @@
> 
>  $(MPLAYER_DIR)/.unpacked: $(DL_DIR)/$(MPLAYER_SOURCE)
>  	$(MPLAYER_CAT) $(DL_DIR)/$(MPLAYER_SOURCE) | tar -C $(BUILD_DIR)
> $(TAR_OPTIONS) -
> -	toolchain/patch-kernel.sh $(MPLAYER_DIR) package/mplayer/
> mplayer-$(MPLAYER_VERSION)\*.patch\*

Seems like your email client wraps long lines.

> +	toolchain/patch-kernel.sh $(MPLAYER_DIR) package/mplayer/
> mplayer-$(MPLAYER_VERSION)\*.patch
> +	toolchain/patch-kernel.sh $(MPLAYER_DIR) package/mplayer/
> mplayer-$(MPLAYER_VERSION)\*.patch.$(ARCH)
> +	$(SED) '/^CFLAGS/s/$$/ -fomit-frame-pointer/'
> $(MPLAYER_DIR)/libavcodec/Makefile
>  	touch $@
> 
>  $(MPLAYER_DIR)/.configured: $(MPLAYER_DIR)/.unpacked

-- 
With kind regards,
Hans-Christian Egtvedt, Applications Engineer