public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [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

* [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

* 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 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

* 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

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).