Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] package/gdb: ARC version has stringent dependencies on host gcc
Date: Thu, 2 Nov 2017 21:41:52 +0000	[thread overview]
Message-ID: <1509658912.7985.7.camel@synopsys.com> (raw)
In-Reply-To: <20171102213815.GF30054@scaer>

Hi Thomas, Yann,

On Thu, 2017-11-02 at 22:38 +0100, Yann E. MORIN wrote:
> Alexey, All,
> 
> On 2017-11-02 21:31 +0000, Alexey Brodkin spake thusly:
> > 
> > On Thu, 2017-11-02 at 19:14 +0100, Yann E. MORIN wrote:
> > > 
> > > The ARC gdb version requires a gcc >= 4.8 (because C++11).
> > > 
> > > Since the host variant uses the same sources as the target variant, we
> > > must ensure that we have a host gcc >= 4.8 as well.
> > > 
> > > Move the architecture dependencies to their own symbol, and also hide
> > > the comment when they are not met.
> > > 
> > > Fixes:
> > > ????https://urldefense.proofpoint.com/v2/url?u=http-3A__autobuild.buildroot.net_results_43eae264991aa369490236c7bd59c0b6a67fcf25_&d=DwIBAg&c=DPL
> > > 6_X_
> > > 6JkXFx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=b-eGjVYMXoKYiImFRBhs00VOY_pLPUoEpAcAm77-
> > > fbo&s=SxA_U3jSTkTx7D0f8Am0c_uZWHh4B6AFeUeYz9ZKe9c&e=
> > > 
> > > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > > Cc: Alexey Brodkin <Alexey.Brodkin@synopsys.com>
> > > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > > ---
> > > ?package/gdb/Config.in.host | 18 ++++++++++++++----
> > > ?1 file changed, 14 insertions(+), 4 deletions(-)
> > > 
> > > diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host
> > > index c2cfc34284..797c07c3f9 100644
> > > --- a/package/gdb/Config.in.host
> > > +++ b/package/gdb/Config.in.host
> > > @@ -1,4 +1,17 @@
> > > +config BR2_PACKAGE_HOST_GDB_ARCH_SUPPORTS
> > > +	bool
> > > +	default y
> > > +	# The ARC version needs C++11, thus gcc >= 4.8, like gdb-8.0.x
> > 
> > Indeed our GDB is based on very up to date upstream sources so all
> > limitations for GDB 8.x are valid for ARC's one.
> > 
> > That said it has nothing to do with ARCH but only GDB version makes sense.
> > So why don't we just introduce BR2_GDB_VERSION_ARC which will be the only option
> > for ARC and will have the same set of "depends" as BR2_GDB_VERSION_8_0?
> 
> If you do that, then you could still select BR2_PACKAGE_HOST_GDB, and
> end up with a choice where there is no entry...
> 
> We can do that in the !ARC case, because there are versions that do not
> require anything, so 8.0 is just hidden and 7.xx are still visible,
> which is not the case for ARC.

Live and learn! :)

Thanks for the explanation.
Indeed GDB situation in BR is not as simple as it looks like from the first sight.

-Alexey

  reply	other threads:[~2017-11-02 21:41 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
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 [this message]
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=1509658912.7985.7.camel@synopsys.com \
    --to=alexey.brodkin@synopsys.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