Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Romain Naour <romain.naour@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] toolchain-external: remove sourcery powerpc toolchains
Date: Sun, 1 May 2016 12:20:58 +0200	[thread overview]
Message-ID: <d8384ec1-051f-d7fe-d7f2-67eb6038ac2d@gmail.com> (raw)
In-Reply-To: <dc8e2f3c-5c65-2d3f-3e9f-4d7fb1625ace@gmail.com>

Hi Gustavo,

Le 30/04/2016 ? 23:57, Romain Naour a ?crit :
> Hi Gustavo,
> 
> Le 29/04/2016 ? 15:18, Gustavo Zacarias a ?crit :
>> These are running long on the teeth - the bundled (e)glibc versions are
>> very old with several security bugs, they don't work reliably with -Os
>> and have several build failures related to internal compiler errors such
>> as:
>>
>> http://autobuild.buildroot.net/results/fe7/fe7bdba5faf199275aedea2918705b5d19d228bf/
>> http://autobuild.buildroot.net/results/935/935ac42c30ed893939c06c077534f060aed80e9a/
>> http://autobuild.buildroot.net/results/a47/a476af82c8fe4a279117314b278b08af9a08fe54/
>> http://autobuild.buildroot.net/results/cae/cae720b5096be2672b4dc1311ae3fc4ed06a3b53/
>>
>> The situation will not provide, and will in fact get worse with older kernel
>> headers precluding modern package versions and the old gcc version doing as
>> well so remove them.
>>
>> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
>> ---
>>  package/liquid-dsp/Config.in                       |  2 -
>>  package/pixman/pixman.mk                           |  5 --
>>  toolchain/toolchain-external/Config.in             | 54 ----------------------
>>  .../toolchain-external/toolchain-external.hash     |  5 --
>>  toolchain/toolchain-external/toolchain-external.mk |  6 ---
>>  5 files changed, 72 deletions(-)
>>
>> diff --git a/package/liquid-dsp/Config.in b/package/liquid-dsp/Config.in
>> index ad887cb..ebdc46e 100644
>> --- a/package/liquid-dsp/Config.in
>> +++ b/package/liquid-dsp/Config.in
>> @@ -1,5 +1,4 @@
>>  comment "liquid-dsp requires a (e)glibc/musl toolchain w/ dynamic library"
>> -	depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC
>>  	depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL) || BR2_STATIC_LIBS
>>  
>>  config BR2_PACKAGE_LIQUID_DSP
>> @@ -7,7 +6,6 @@ config BR2_PACKAGE_LIQUID_DSP
>>  	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL
>>  	depends on !BR2_STATIC_LIBS
>>  	# These result in internal compiler error
> 
> This comment should be removed.

Also, add a new entry for BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC and
BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC_E500V2 in Config.in.legacy

Best regards,
Romain

> 
> With that fixed:
> 
> Reviewed-by: Romain Naour <romain.naour@gmail.com>
> 
> Best regards,
> Romain
> 

  reply	other threads:[~2016-05-01 10:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-29 13:18 [Buildroot] [PATCH] toolchain-external: remove sourcery powerpc toolchains Gustavo Zacarias
2016-04-30 21:57 ` Romain Naour
2016-05-01 10:20   ` Romain Naour [this message]
2016-05-01 13:06 ` Thomas Petazzoni

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=d8384ec1-051f-d7fe-d7f2-67eb6038ac2d@gmail.com \
    --to=romain.naour@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