Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] package/gdb: ARC version has stringent dependencies on host gcc
Date: Thu, 2 Nov 2017 22:31:30 +0100	[thread overview]
Message-ID: <20171102223130.5354ecd5@windsurf> (raw)
In-Reply-To: <20171102212543.GE30054@scaer>

Hello,

On Thu, 2 Nov 2017 22:25:43 +0100, Yann E. MORIN wrote:

> > Why do we have a dependency on the target compiler version for
> > host-gdb ?  
> 
> For two reasons:
> 
> 1. you said so in your analysis. OK, you could be wrong, so...
> 2. that is the case for gdb-8.0.x, later in that same file.

I think I know the logic for gdb 8.x. The logic is that:

 * If you chose only target gdb (and not host gdb), you don't have a
   version choice, and we currently use gdb 7.11, which doesn't have a
   compiler version requirement.

 * If you chose to build both host gdb and target gdb, then the
   selection of the gdb version for the host gdb affects the version
   used to build target gdb. I.e, you select gdb 8.0 as the host-gdb,
   and gdb 8.0 will also be built as the target gdb. Therefore we need
   to ensure that the host gdb 8.0 option can only be selected if both
   the host *and* the target compiler are >= 4.8.

That is a bit tricky, and would warrant a comment in the Config.in file
I guess.

But anyway, this means your patch is correct! :)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2017-11-02 21:31 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-02 18:14 [Buildroot] [PATCH] package/gdb: ARC version has stringent dependencies on host gcc Yann E. MORIN
2017-11-02 21:12 ` Thomas Petazzoni
2017-11-02 21:25   ` Yann E. MORIN
2017-11-02 21:31     ` Thomas Petazzoni [this message]
2017-11-02 21:31 ` Alexey Brodkin
2017-11-02 21:37   ` Thomas Petazzoni
2017-11-02 21:38   ` Yann E. MORIN
2017-11-02 21:41     ` Alexey Brodkin
2017-11-02 21:44 ` Thomas Petazzoni
2017-11-02 22:07   ` Alexey Brodkin

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=20171102223130.5354ecd5@windsurf \
    --to=thomas.petazzoni@free-electrons.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