public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [pushed/ob] Change gdb/version.in to 9.0.50.DATE-git (new version numbering scheme)
@ 2019-10-06 15:35 Joel Brobecker
  2019-10-07 10:54 ` [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp Tom de Vries
  0 siblings, 1 reply; 6+ messages in thread
From: Joel Brobecker @ 2019-10-06 15:35 UTC (permalink / raw)
  To: gdb-patches; +Cc: Joel Brobecker

Hi everyone,

As discussed and then announced, the numbering scheme used for
future GDB releases is changed. I therefore modified gdb/version.in
to reflect the new scheme.

Next up for me: Update the documentation on the wiki.

gdb/ChangeLog:

        * version.in: Change version number to "9.0.50.DATE-git".
---
 gdb/ChangeLog  | 4 ++++
 gdb/version.in | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4eeed4931e7..7bfcab593b8 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2019-10-06  Joel Brobecker  <brobecker@adacore.com>
+
+	* version.in: Change version number to "9.0.50.DATE-git".
+
 2019-10-03  Tom Tromey  <tom@tromey.com>
 
 	PR rust/24976:
diff --git a/gdb/version.in b/gdb/version.in
index 0ab8049edb0..bf3348a01e0 100644
--- a/gdb/version.in
+++ b/gdb/version.in
@@ -1 +1 @@
-8.3.50.DATE-git
+9.0.50.DATE-git
-- 
2.17.1

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp
  2019-10-06 15:35 [pushed/ob] Change gdb/version.in to 9.0.50.DATE-git (new version numbering scheme) Joel Brobecker
@ 2019-10-07 10:54 ` Tom de Vries
  2019-10-07 17:17   ` Tom Tromey
  0 siblings, 1 reply; 6+ messages in thread
From: Tom de Vries @ 2019-10-07 10:54 UTC (permalink / raw)
  To: Joel Brobecker, gdb-patches

[-- Attachment #1: Type: text/plain, Size: 449 bytes --]

[ was: Re: [pushed/ob] Change gdb/version.in to 9.0.50.DATE-git (new
version numbering scheme) ]
On 06-10-19 17:35, Joel Brobecker wrote:

> Hi everyone,
> 
> As discussed and then announced, the numbering scheme used for
> future GDB releases is changed. I therefore modified gdb/version.in
> to reflect the new scheme.
> 

Hi,

I noticed a regression in default.exp due to this update.

Fixed by commit below, committed as obvious.

Thanks,
- Tom

[-- Attachment #2: 0001-gdb-testsuite-Update-expected-_gdb_major-_gdb_minor-in-default.exp.patch --]
[-- Type: text/x-patch, Size: 1156 bytes --]

[gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp

Now that commit "225f296a023 Change gdb/version.in to 9.0.50.DATE-git (new
version numbering scheme)" has changed the gdb version number, we see:
...
FAIL: gdb.base/default.exp: show convenience ($_gdb_major = 8 not found)
...

Fix this by updating the expected _gdb_major/_gdb_minor to 9.1.

Tested on x86_64-linux.

gdb/testsuite/ChangeLog:

2019-10-07  Tom de Vries  <tdevries@suse.de>

	* gdb.base/default.exp: Expect _gdb_major/_gdb_minor to be 9.1.

---
 gdb/testsuite/gdb.base/default.exp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gdb/testsuite/gdb.base/default.exp b/gdb/testsuite/gdb.base/default.exp
index 0325b8045d3..cf009cdc771 100644
--- a/gdb/testsuite/gdb.base/default.exp
+++ b/gdb/testsuite/gdb.base/default.exp
@@ -604,8 +604,8 @@ set show_conv_list \
 	{$_cimag = <internal function _cimag>} \
 	{$_creal = <internal function _creal>} \
 	{$_isvoid = <internal function _isvoid>} \
-	{$_gdb_major = 8} \
-	{$_gdb_minor = 4} \
+	{$_gdb_major = 9} \
+	{$_gdb_minor = 1} \
 	{$_shell_exitsignal = void} \
 	{$_shell_exitcode = 0} \
     }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp
  2019-10-07 10:54 ` [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp Tom de Vries
@ 2019-10-07 17:17   ` Tom Tromey
  2019-10-08  1:53     ` Joel Brobecker
  0 siblings, 1 reply; 6+ messages in thread
From: Tom Tromey @ 2019-10-07 17:17 UTC (permalink / raw)
  To: Tom de Vries; +Cc: Joel Brobecker, gdb-patches

>>>>> "Tom" == Tom de Vries <tdevries@suse.de> writes:

Tom> I noticed a regression in default.exp due to this update.

Tom> Fixed by commit below, committed as obvious.

Is there some version-bumping script that must also be updated?

Tom

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp
  2019-10-07 17:17   ` Tom Tromey
@ 2019-10-08  1:53     ` Joel Brobecker
  2019-10-08 20:57       ` Tom Tromey
  0 siblings, 1 reply; 6+ messages in thread
From: Joel Brobecker @ 2019-10-08  1:53 UTC (permalink / raw)
  To: Tom Tromey; +Cc: Tom de Vries, gdb-patches

> >>>>> "Tom" == Tom de Vries <tdevries@suse.de> writes:
> 
> Tom> I noticed a regression in default.exp due to this update.
> 
> Tom> Fixed by commit below, committed as obvious.

Thanks for doing that, Tom. I'm going to have to start thinking
about what to do for future branches. We are all going to get
annoyed if we need to bump this each time, particuarly if I keep
forgetting!

> Is there some version-bumping script that must also be updated?

You mean the nightly bump? If yes, that one should be fine,
since the format is actually the same as before. In essence,
the only thing that changes is the fact that we bump the major
number rather than the minor one.

I was really surprised how easy it was to adapt my scripts to
the new scheme until I came to realize that the new scheme is
actually very consistent with the old one, with that minor but
important distinction.

-- 
Joel

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp
  2019-10-08  1:53     ` Joel Brobecker
@ 2019-10-08 20:57       ` Tom Tromey
  2019-10-09  2:35         ` Joel Brobecker
  0 siblings, 1 reply; 6+ messages in thread
From: Tom Tromey @ 2019-10-08 20:57 UTC (permalink / raw)
  To: Joel Brobecker; +Cc: Tom Tromey, Tom de Vries, gdb-patches

>> Is there some version-bumping script that must also be updated?

Joel> You mean the nightly bump?

No, I meant if you have a script to do the post-branch bump, does it
need to be updated to fix up default.exp.

Tom

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp
  2019-10-08 20:57       ` Tom Tromey
@ 2019-10-09  2:35         ` Joel Brobecker
  0 siblings, 0 replies; 6+ messages in thread
From: Joel Brobecker @ 2019-10-09  2:35 UTC (permalink / raw)
  To: Tom Tromey; +Cc: Tom de Vries, gdb-patches

> >> Is there some version-bumping script that must also be updated?
> 
> Joel> You mean the nightly bump?
> 
> No, I meant if you have a script to do the post-branch bump, does it
> need to be updated to fix up default.exp.

I wouldn't do it automatically. What I could be doing is add a reminder
to do it. But then, I'd have to test the change, as you never know
what I might be doing with those fat fingers of mine (and my evil
twin standing behind me begging to play with my keyboard and mouse).

I'm thinking instead we should change the test to be immune from
those version bumps.

Could we make that test verify that the minor number is zero, and
that the major number looks like a number, maybe?

Otherwise, how about extracting the version number from the output
of "show version" and then use that to validate the test(s)?

-- 
Joel

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-10-09  2:35 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-06 15:35 [pushed/ob] Change gdb/version.in to 9.0.50.DATE-git (new version numbering scheme) Joel Brobecker
2019-10-07 10:54 ` [committed][gdb/testsuite] Update expected _gdb_major/_gdb_minor in default.exp Tom de Vries
2019-10-07 17:17   ` Tom Tromey
2019-10-08  1:53     ` Joel Brobecker
2019-10-08 20:57       ` Tom Tromey
2019-10-09  2:35         ` Joel Brobecker

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