Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Bernhard Fischer <rep.dot.nop@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] mplayer-package doesn't build for	otherarchitectures than avr32
Date: Fri, 31 Aug 2007 09:31:56 +0200	[thread overview]
Message-ID: <20070831073156.GA29704@aon.at> (raw)
In-Reply-To: <014601c7eb5a$138e37e0$36f2780a@atmel.com>

On Fri, Aug 31, 2007 at 12:30:28AM +0200, Ulf Samuelsson wrote:
>----- Original Message ----- 
>From: "Bernhard Fischer" <rep.dot.nop@gmail.com>
>To: "Simon Pasch" <fpasch@googlemail.com>
>Cc: <buildroot@uclibc.org>; <ulf@atmel.com>
>Sent: Friday, August 31, 2007 12:14 AM
>Subject: Re: [Buildroot] mplayer-package doesn't build for otherarchitectures than avr32
>
>
>> On Wed, Aug 29, 2007 at 10:33:04AM +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 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).
>>>
>>>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\*
>>>+ 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
>> 
>> Ulf, can you fix this up in a sane manner, i.e. not add ...$(ARCH).patch
>> stuff but arch agnostic, universally working fixes, if any?
>> 
>> TIA,
>>
>
>Got it from Hans Christian, which in his turn, got it from a colleague,
>so they need to update it if that is to happen.
>A lot of the stuff seems to be AVR32 specific in-line assembly code 
>using the AVR32 DSP features to optimize the mplayer algorithms, 
>so it seems utterly useless to apply for other architectures.

Given that mplayer currently is at 1.0-rc1 (i.e. not released), what
about fixing this in the proper place, i.e. upstream?
$ svn ls svn://svn.mplayerhq.hu/ffmpeg/trunk/libavcodec/ | grep -i avr
$
Turns up nothing..

  reply	other threads:[~2007-08-31  7:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-29  8:33 [Buildroot] mplayer-package doesn't build for other architectures than avr32 Simon Pasch
2007-08-29  8:48 ` Hans-Christian Egtvedt
2007-08-30 22:14 ` Bernhard Fischer
2007-08-30 22:30   ` [Buildroot] mplayer-package doesn't build for otherarchitectures " Ulf Samuelsson
2007-08-31  7:31     ` Bernhard Fischer [this message]
2007-08-31 16:05       ` [Buildroot] mplayer-package doesn't build forotherarchitectures " Ulf Samuelsson
2007-09-01 16:31         ` Bernhard Fischer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070831073156.GA29704@aon.at \
    --to=rep.dot.nop@gmail.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox