From: Peter Korsgaard <peter@korsgaard.com>
To: Alexey Brodkin via buildroot <buildroot@buildroot.org>
Cc: Alexey Brodkin <Alexey.Brodkin@synopsys.com>,
"Yann E. MORIN" <yann.morin.1998@free.fr>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Subject: Re: [Buildroot] [PATCH] package/gdb: Remove special handling of libstdc++ for ARC
Date: Fri, 12 Jan 2024 17:34:43 +0100 [thread overview]
Message-ID: <87bk9q1pek.fsf@48ers.dk> (raw)
In-Reply-To: <DS7PR12MB90419624B8739BAF541F133DA161A@DS7PR12MB9041.namprd12.prod.outlook.com> (Alexey Brodkin via buildroot's message of "Tue, 2 Jan 2024 21:55:13 +0000")
>>>>> "Alexey" == Alexey Brodkin via buildroot <buildroot@buildroot.org> writes:
> Hi Yann,
>> On 2024-01-02 07:55 -0800, Alexey Brodkin via buildroot spake thusly:
>> > Effectively that's a revert of a very old fix [1]
>> > which is no longer needed.
>>
>> It would have been nice to get a better indication of how much time it
>> has not been needed anymore, so that we know wehther we should backport
>> that to maintenance beranches. ca99d0ea925e5a8 mentions that it would be
>> fixed in 2017.03, so we can certainly assume that it has been fixed for
>> quite some time now, and thus we can backport this to all our
>> maintenance branches (2023.02 LTS and 2023.11).
>>
>> I forgot to add that information befoe pushing, sorry.
> Well, indeed it's been so many years since that fix and I'm sure
> that problem is long gone.
> What's moreinteresting, that option "--disable-build-with-cxx" was removed
> in GDB 8.x in September 2016 as since then C++ is required, see
> https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=cf6de44d75082116865a85cbf94db2632b679361
> That said it's totaly fine to back-port that fix to stable Buildroot branches,
> but since it's not functional (it doesn't change anyting in reality) I'd say
> there's no point in back-porting it.
Committed to 2023.02.x and 2023.11.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2024-01-12 16:34 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-02 15:55 [Buildroot] [PATCH] package/gdb: Remove special handling of libstdc++ for ARC Alexey Brodkin via buildroot
2024-01-02 20:46 ` Yann E. MORIN
2024-01-02 21:55 ` Alexey Brodkin via buildroot
2024-01-03 20:31 ` Yann E. MORIN
2024-01-12 16:34 ` Peter Korsgaard [this message]
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=87bk9q1pek.fsf@48ers.dk \
--to=peter@korsgaard.com \
--cc=Alexey.Brodkin@synopsys.com \
--cc=buildroot@buildroot.org \
--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.