Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 2/2] binutils: remove support for version 2.23
Date: Fri, 22 Apr 2016 00:13:24 +0200	[thread overview]
Message-ID: <57195084.7040504@mind.be> (raw)
In-Reply-To: <1461186614-2391-2-git-send-email-thomas.petazzoni@free-electrons.com>

On 04/20/16 23:10, Thomas Petazzoni wrote:
> Now that we have added support for the lastest binutils version 2.26,
> it is time to remove the oldest version we currently support, 2.23, in
> order to keep only 3 versions: 2.24, 2.25 and 2.26.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
> Changes since v1:
>    - Remove hash from the hash file, as noticed by Romain Naour.
>    - Add Config.in.legacy entry, as noticed by Romain Naour.
> ---
[snip]
> diff --git a/Config.in.legacy b/Config.in.legacy
> index ba3d94c..f52d5f8 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -145,6 +145,13 @@ endif
>   ###############################################################################
>   comment "Legacy options removed in 2016.05"
>
> +config BR2_BINUTILS_VERSION_2_23_X
> +	bool "binutils 2.23 removed"
> +	select BR2_LEGACY
> +	help
> +	  Binutils 2.23 has been removed, using a newer version is
> +	  recommended.

  I don't know if it very important, but it's a bit weird that another legacy 
option (BR2_BINUTILS_VERSION_2_23_1) still selects this one. So ideally that 
select should be removed, and its help text updated to be the same as this one.


  Regards,
  Arnout

> +
>   config BR2_x86_i386
>   	bool "x86 i386 support removed"
>   	help

[snip]


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

  reply	other threads:[~2016-04-21 22:13 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-20 21:10 [Buildroot] [PATCH v2 1/2] binutils: remove 2.22 hash Thomas Petazzoni
2016-04-20 21:10 ` [Buildroot] [PATCH v2 2/2] binutils: remove support for version 2.23 Thomas Petazzoni
2016-04-21 22:13   ` Arnout Vandecappelle [this message]
2016-04-21 22:10 ` [Buildroot] [PATCH v2 1/2] binutils: remove 2.22 hash Arnout Vandecappelle
2016-04-22 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=57195084.7040504@mind.be \
    --to=arnout@mind.be \
    --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