public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug cli/29945] New: "set debug timestamp" broken
@ 2022-12-27 20:26 tromey at sourceware dot org
  2022-12-27 20:32 ` [Bug cli/29945] " tromey at sourceware dot org
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: tromey at sourceware dot org @ 2022-12-27 20:26 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

            Bug ID: 29945
           Summary: "set debug timestamp" broken
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: cli
          Assignee: unassigned at sourceware dot org
          Reporter: tromey at sourceware dot org
  Target Milestone: ---

I broke "set debug timestamp 1" in:

commit b8043d27217ff89abba733476cb71c3656f5722a
Author: Tom Tromey <tromey@adacore.com>
Date:   Thu Aug 11 13:16:46 2022 -0600

    Remove a ui-related memory leak


Reverting this makes it work again, but maybe there's a better fix.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
@ 2022-12-27 20:32 ` tromey at sourceware dot org
  2022-12-27 21:49 ` tromey at sourceware dot org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: tromey at sourceware dot org @ 2022-12-27 20:32 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|HEAD                        |13.1

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
  2022-12-27 20:32 ` [Bug cli/29945] " tromey at sourceware dot org
@ 2022-12-27 21:49 ` tromey at sourceware dot org
  2022-12-27 23:40 ` tromey at sourceware dot org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: tromey at sourceware dot org @ 2022-12-27 21:49 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

--- Comment #1 from Tom Tromey <tromey at sourceware dot org> ---
I have a better patch but haven't written a test case yet.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
  2022-12-27 20:32 ` [Bug cli/29945] " tromey at sourceware dot org
  2022-12-27 21:49 ` tromey at sourceware dot org
@ 2022-12-27 23:40 ` tromey at sourceware dot org
  2022-12-28 16:37 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: tromey at sourceware dot org @ 2022-12-27 23:40 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at sourceware dot org   |tromey at sourceware dot org

--- Comment #2 from Tom Tromey <tromey at sourceware dot org> ---
https://sourceware.org/pipermail/gdb-patches/2022-December/195117.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
                   ` (2 preceding siblings ...)
  2022-12-27 23:40 ` tromey at sourceware dot org
@ 2022-12-28 16:37 ` cvs-commit at gcc dot gnu.org
  2022-12-28 16:48 ` cvs-commit at gcc dot gnu.org
  2022-12-28 17:04 ` tromey at sourceware dot org
  5 siblings, 0 replies; 7+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-12-28 16:37 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom Tromey <tromey@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a60535c39ba52d88c47740db6ab116db32e2331a

commit a60535c39ba52d88c47740db6ab116db32e2331a
Author: Tom Tromey <tom@tromey.com>
Date:   Tue Dec 27 16:34:44 2022 -0700

    Fix "set debug timestamp"

    PR cli/29945 points out that "set debug timestamp 1" stopped working
    -- this is a regression due to commit b8043d27 ("Remove a ui-related
    memory leak").

    This patch fixes the bug and adds a regression test.

    I think this should probably be backported to the gdb 13 branch.

    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29945

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
                   ` (3 preceding siblings ...)
  2022-12-28 16:37 ` cvs-commit at gcc dot gnu.org
@ 2022-12-28 16:48 ` cvs-commit at gcc dot gnu.org
  2022-12-28 17:04 ` tromey at sourceware dot org
  5 siblings, 0 replies; 7+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-12-28 16:48 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The gdb-13-branch branch has been updated by Tom Tromey
<tromey@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ecb7214fc18f63693dac6065e36802e0677412e7

commit ecb7214fc18f63693dac6065e36802e0677412e7
Author: Tom Tromey <tom@tromey.com>
Date:   Tue Dec 27 16:34:44 2022 -0700

    Fix "set debug timestamp"

    PR cli/29945 points out that "set debug timestamp 1" stopped working
    -- this is a regression due to commit b8043d27 ("Remove a ui-related
    memory leak").

    This patch fixes the bug and adds a regression test.

    I think this should probably be backported to the gdb 13 branch.

    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29945

    (cherry picked from commit a60535c39ba52d88c47740db6ab116db32e2331a)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

* [Bug cli/29945] "set debug timestamp" broken
  2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
                   ` (4 preceding siblings ...)
  2022-12-28 16:48 ` cvs-commit at gcc dot gnu.org
@ 2022-12-28 17:04 ` tromey at sourceware dot org
  5 siblings, 0 replies; 7+ messages in thread
From: tromey at sourceware dot org @ 2022-12-28 17:04 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=29945

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |13.1
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #5 from Tom Tromey <tromey at sourceware dot org> ---
Fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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

end of thread, other threads:[~2022-12-28 17:05 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-27 20:26 [Bug cli/29945] New: "set debug timestamp" broken tromey at sourceware dot org
2022-12-27 20:32 ` [Bug cli/29945] " tromey at sourceware dot org
2022-12-27 21:49 ` tromey at sourceware dot org
2022-12-27 23:40 ` tromey at sourceware dot org
2022-12-28 16:37 ` cvs-commit at gcc dot gnu.org
2022-12-28 16:48 ` cvs-commit at gcc dot gnu.org
2022-12-28 17:04 ` tromey at sourceware dot org

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