All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Arnout Vandecappelle <arnout@mind.be>
Cc: Giulio Benetti <giulio.benetti@benettiengineering.com>,
	Romain Naour <romain.naour@gmail.com>,
	"Yann E. MORIN" <yann.morin.1998@free.fr>,
	Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>,
	Buildroot List <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH v2 07/14] toolchain/toolchain-external: add BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER option
Date: Tue, 14 Jun 2022 17:58:44 +0200	[thread overview]
Message-ID: <20220614175844.45509426@windsurf> (raw)
In-Reply-To: <12178427-e595-be4e-5da5-d4cf1718b9e2@mind.be>

On Tue, 14 Jun 2022 17:34:20 +0200
Arnout Vandecappelle <arnout@mind.be> wrote:

>   This could have been simplified to
> 
> 	depends on BR2_PACKAGE_GDB_ARCH_SUPPORTS
> 
> but this way is more flexible so better I guess.

Yes, I thought about this (should maybe have written it in the commit
log), but I thought: external toolchains can very well rely on patched
GDB versions that could potentially have support for an architecture
that's not supported in upstream GDB.

I agree this is a corner case, so if we decide that's not worth
considering, I would totally support using
BR2_PACKAGE_GDB_ARCH_SUPPORTS.

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2022-06-14 15:59 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-08  7:10 [Buildroot] [PATCH v2 00/14] Another set of Bootlin toolchain integration improvements Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 01/14] support/scripts/gen-bootlin-toolchains: fix RISC-V 64-bit toolchain description Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 02/14] support/testing, toolchain/toolchain-external/toolchain-external-bootlin: regenerate with latest gen-bootlin-toolchains script Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 03/14] support/scripts/gen-bootlin-toolchains: improve dependencies of ARM toolchains Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 04/14] toolchain/toolchain-external/toolchain-external-bootlin: regenerate with BR2_ARM dependency Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 05/14] support/scripts/gen-bootlin-toolchains: adjust dependencies of i686 toolchains Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 06/14] toolchain/toolchain-external/toolchain-external-bootlin: regenerate after i686 toolchain dependency fixes Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 07/14] toolchain/toolchain-external: add BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER option Thomas Petazzoni via buildroot
2022-06-14 15:34   ` Arnout Vandecappelle
2022-06-14 15:58     ` Thomas Petazzoni via buildroot [this message]
2022-06-14 16:56       ` Arnout Vandecappelle
2022-06-14 21:41         ` Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 08/14] support/scripts/gen-bootlin-toolchains: make use of BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 09/14] toolchain/toolchain-external/toolchain-external-bootlin: regenerate with BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 10/14] support/scripts/gen-bootlin-toolchains: check that toolchains exists for a certain arch Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 11/14] support/scripts/gen-bootlin-toolchains: drop "mips64" architecture variant Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 12/14] toolchain/toolchain-external/toolchain-external-bootlin: regenerate after mips64 toolchain removal Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 13/14] support/scripts/gen-bootlin-toolchains: add support for ARMv7 big endian toolchain Thomas Petazzoni via buildroot
2022-06-08  7:10 ` [Buildroot] [PATCH v2 14/14] support/testing, toolchain/toolchain-external/toolchain-external-bootlin: regenerate with " Thomas Petazzoni via buildroot
2022-06-14 15:43 ` [Buildroot] [PATCH v2 00/14] Another set of Bootlin toolchain integration improvements Arnout Vandecappelle

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=20220614175844.45509426@windsurf \
    --to=buildroot@buildroot.org \
    --cc=arnout@mind.be \
    --cc=giulio.benetti@benettiengineering.com \
    --cc=romain.naour@gmail.com \
    --cc=thomas.de_schampheleire@nokia.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=yann.morin.1998@free.fr \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.