[Bug testsuite/24831] New: FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug testsuite/24831] New: FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

986882896 at qq dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24831

            Bug ID: 24831
           Summary: FAIL: gdb.multi/tids.exp: two inferiors: info threads
                    -1
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: testsuite
          Assignee: unassigned at sourceware dot org
          Reporter: vries at gcc dot gnu.org
  Target Milestone: ---

I observe these fails:
...
FAIL: gdb.multi/tids.exp: two inferiors: info threads -1
FAIL: gdb.multi/tids.exp: two inferiors: info threads -$one
...

In more detail:
...
(gdb) PASS: gdb.multi/tids.exp: two inferiors: thread apply 2-$one
info threads -1^M
Unrecognized option at: -1^M
(gdb) FAIL: gdb.multi/tids.exp: two inferiors: info threads -1
thread apply -1^M
Invalid thread ID: -1^M
(gdb) PASS: gdb.multi/tids.exp: two inferiors: thread apply -1
info threads -$one^M
Unrecognized option at: -$one^M
(gdb) FAIL: gdb.multi/tids.exp: two inferiors: info threads -$one
...

Also observed here: https://sourceware.org/ml/gdb-testers/2019-q3/msg00316.html
.

Fallout from 54d6600669 'Make "info threads" use the gdb::option framework'

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug testsuite/24831] FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

986882896 at qq dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24831

--- Comment #1 from Tom de Vries <vries at gcc dot gnu.org> ---
Created attachment 11916
  --> https://sourceware.org/bugzilla/attachment.cgi?id=11916&action=edit
Tentative patch

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug testsuite/24831] FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

986882896 at qq dot com
In reply to this post by 986882896 at qq dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24831

Tom de Vries <vries at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |palves at redhat dot com

--- Comment #2 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Tom de Vries from comment #1)
> Created attachment 11916 [details]
> Tentative patch

Pedro, WDYT?

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug testsuite/24831] FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

986882896 at qq dot com
In reply to this post by 986882896 at qq dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24831

--- 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 de Vries <[hidden email]>:

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

commit 024a5840006aba9ba698aa9d868a7394016dd4a0
Author: Tom de Vries <[hidden email]>
Date:   Wed Jul 24 08:36:17 2019 +0200

    Update expected info threads error messages in gdb.multi/tids.exp

    We currently have these FAILs:
    ...
    FAIL: gdb.multi/tids.exp: two inferiors: info threads -1
    FAIL: gdb.multi/tids.exp: two inferiors: info threads -$one
    ...
    because we're expecting:
    ...
    Invalid thread ID: -1
    ...
    but instead we have:
    ...
    Unrecognized option at: -1
    ...

    This error message for info threads has changed since commit 54d6600669
    'Make "info threads" use the gdb::option framework'.

    Update the test accordingly.

    Tested on x86_64-linux.

    gdb/testsuite/ChangeLog:

    2019-07-24  Tom de Vries  <[hidden email]>

        PR testsuite/24831
        * gdb.multi/tids.exp: Update error messages for info threads.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug testsuite/24831] FAIL: gdb.multi/tids.exp: two inferiors: info threads -1

986882896 at qq dot com
In reply to this post by 986882896 at qq dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24831

Tom de Vries <vries at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Tom de Vries <vries at gcc dot gnu.org> ---
Patch committed, marking resolved-fixed.

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