* [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default
@ 2016-08-03 14:49 Pedro Alves
2016-08-03 14:49 ` [PATCH+7.12 1/1] " Pedro Alves
2016-08-03 16:30 ` [PATCH+7.12 0/1] " Yao Qi
0 siblings, 2 replies; 6+ messages in thread
From: Pedro Alves @ 2016-08-03 14:49 UTC (permalink / raw)
To: gdb-patches
I noticed that the 7.12 NEWS does not mention yet that C++ is now the
default. Might be a good idea to mention it, and suggest how to fall
back to C, if necessary, taking the opportunity to warn that the
option will be removed in the future.
Follows a patch for that. OK?
Pedro Alves (1):
gdb/NEWS: Mention C++ is now the default
gdb/NEWS | 7 +++++++
1 file changed, 7 insertions(+)
--
2.5.5
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH+7.12 1/1] gdb/NEWS: Mention C++ is now the default
2016-08-03 14:49 [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default Pedro Alves
@ 2016-08-03 14:49 ` Pedro Alves
2016-08-03 16:25 ` Eli Zaretskii
2016-08-03 16:30 ` [PATCH+7.12 0/1] " Yao Qi
1 sibling, 1 reply; 6+ messages in thread
From: Pedro Alves @ 2016-08-03 14:49 UTC (permalink / raw)
To: gdb-patches
gdb/ChangeLog:
yyyy-mm-dd Pedro Alves <palves@redhat.com>
* NEWS: Mention that GDB and GDBserver build with a C++ compiler by
default.
---
gdb/NEWS | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/gdb/NEWS b/gdb/NEWS
index 4d72027..43737cb 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -5,6 +5,13 @@
*** Changes in GDB 7.12
+* GDB and GDBserver now build with a C++ compiler by default.
+
+ The --enable-build-with-cxx configure option now defaults to "yes".
+ One must now explicitly configure with --enable-build-with-cxx=no in
+ order to build with a C compiler. This option will be removed in a
+ future release.
+
* GDBserver now supports recording btrace without maintaining an active
GDB connection.
--
2.5.5
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH+7.12 1/1] gdb/NEWS: Mention C++ is now the default
2016-08-03 14:49 ` [PATCH+7.12 1/1] " Pedro Alves
@ 2016-08-03 16:25 ` Eli Zaretskii
2016-08-05 16:16 ` Pedro Alves
0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2016-08-03 16:25 UTC (permalink / raw)
To: Pedro Alves; +Cc: gdb-patches
> From: Pedro Alves <palves@redhat.com>
> Date: Wed, 3 Aug 2016 15:49:10 +0100
>
> diff --git a/gdb/NEWS b/gdb/NEWS
> index 4d72027..43737cb 100644
> --- a/gdb/NEWS
> +++ b/gdb/NEWS
> @@ -5,6 +5,13 @@
>
> *** Changes in GDB 7.12
>
> +* GDB and GDBserver now build with a C++ compiler by default.
> +
> + The --enable-build-with-cxx configure option now defaults to "yes".
> + One must now explicitly configure with --enable-build-with-cxx=no in
> + order to build with a C compiler. This option will be removed in a
> + future release.
> +
> * GDBserver now supports recording btrace without maintaining an active
> GDB connection.
The text is okay, but I think "./configure --help" should mention
"--disable-build-with-cxx", as it does with all the other options that
are enabled by default.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH+7.12 1/1] gdb/NEWS: Mention C++ is now the default
2016-08-03 16:25 ` Eli Zaretskii
@ 2016-08-05 16:16 ` Pedro Alves
0 siblings, 0 replies; 6+ messages in thread
From: Pedro Alves @ 2016-08-05 16:16 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: gdb-patches
On 08/03/2016 05:25 PM, Eli Zaretskii wrote:
> The text is okay, but I think "./configure --help" should mention
> "--disable-build-with-cxx", as it does with all the other options that
> are enabled by default.
>
That's now done, so here's what I'm pushing to master and 7.12,
updated to talk about --disable-build-with-cxx.
From 71cf9b4a255d120100391d1f661fbaf556fccea7 Mon Sep 17 00:00:00 2001
From: Pedro Alves <palves@redhat.com>
Date: Fri, 5 Aug 2016 17:09:31 +0100
Subject: [PATCH] gdb/NEWS: Mention that C++ is now the default
gdb/ChangeLog:
2016-08-05 Pedro Alves <palves@redhat.com>
* NEWS: Mention that GDB and GDBserver build with a C++ compiler by
default.
---
gdb/NEWS | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/gdb/NEWS b/gdb/NEWS
index 4d72027..b08d8a0 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -5,6 +5,13 @@
*** Changes in GDB 7.12
+* GDB and GDBserver now build with a C++ compiler by default.
+
+ The --enable-build-with-cxx configure option is now enabled by
+ default. One must now explicitly configure with
+ --disable-build-with-cxx in order to build with a C compiler. This
+ option will be removed in a future release.
+
* GDBserver now supports recording btrace without maintaining an active
GDB connection.
--
2.5.5
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default
2016-08-03 14:49 [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default Pedro Alves
2016-08-03 14:49 ` [PATCH+7.12 1/1] " Pedro Alves
@ 2016-08-03 16:30 ` Yao Qi
2016-08-03 17:52 ` Pedro Alves
1 sibling, 1 reply; 6+ messages in thread
From: Yao Qi @ 2016-08-03 16:30 UTC (permalink / raw)
To: Pedro Alves; +Cc: gdb-patches
On Wed, Aug 3, 2016 at 3:49 PM, Pedro Alves <palves@redhat.com> wrote:
> I noticed that the 7.12 NEWS does not mention yet that C++ is now the
> default. Might be a good idea to mention it, and suggest how to fall
> back to C, if necessary, taking the opportunity to warn that the
> option will be removed in the future.
Yes, good to mention it. Does "+7.12" in subject mean the patch is for both
master and 7.12? or only for 7.12. My understand is the former.
--
Yao (齐尧)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default
2016-08-03 16:30 ` [PATCH+7.12 0/1] " Yao Qi
@ 2016-08-03 17:52 ` Pedro Alves
0 siblings, 0 replies; 6+ messages in thread
From: Pedro Alves @ 2016-08-03 17:52 UTC (permalink / raw)
To: Yao Qi; +Cc: gdb-patches
On 08/03/2016 05:30 PM, Yao Qi wrote:
> On Wed, Aug 3, 2016 at 3:49 PM, Pedro Alves <palves@redhat.com> wrote:
>> I noticed that the 7.12 NEWS does not mention yet that C++ is now the
>> default. Might be a good idea to mention it, and suggest how to fall
>> back to C, if necessary, taking the opportunity to warn that the
>> option will be removed in the future.
>
> Yes, good to mention it. Does "+7.12" in subject mean the patch is for both
> master and 7.12? or only for 7.12. My understand is the former.
Yesh, both master and 7.12.
Thanks,
Pedro Alves
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-08-05 16:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-03 14:49 [PATCH+7.12 0/1] gdb/NEWS: Mention C++ is now the default Pedro Alves
2016-08-03 14:49 ` [PATCH+7.12 1/1] " Pedro Alves
2016-08-03 16:25 ` Eli Zaretskii
2016-08-05 16:16 ` Pedro Alves
2016-08-03 16:30 ` [PATCH+7.12 0/1] " Yao Qi
2016-08-03 17:52 ` Pedro Alves
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).