diff mbox series

Add gdb.threads/thread_events.exp to run in AIX as well

Message ID 20240429014855.3031-1-akamath996@gmail.com
State New
Headers
Series Add gdb.threads/thread_events.exp to run in AIX as well |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_gdb_build--master-arm success Testing passed
linaro-tcwg-bot/tcwg_gdb_check--master-arm success Testing passed
linaro-tcwg-bot/tcwg_gdb_check--master-aarch64 success Testing passed

Commit Message

Aditya Vidyadhar Kamath April 29, 2024, 1:48 a.m. UTC 
  From: Aditya Vidyadhar Kamath <Aditya.Kamath1@ibm.com>

---
 gdb/testsuite/gdb.threads/thread_events.exp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Kevin Buettner May 2, 2024, 1:57 a.m. UTC | #1 
On Mon, 29 Apr 2024 07:18:57 +0530
Aditya Vidyadhar Kamath <akamath996@gmail.com> wrote:

> From: Aditya Vidyadhar Kamath <Aditya.Kamath1@ibm.com>
> 
> ---
>  gdb/testsuite/gdb.threads/thread_events.exp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/gdb/testsuite/gdb.threads/thread_events.exp b/gdb/testsuite/gdb.threads/thread_events.exp
> index 100205e5f58..11a89139384 100644
> --- a/gdb/testsuite/gdb.threads/thread_events.exp
> +++ b/gdb/testsuite/gdb.threads/thread_events.exp
> @@ -23,7 +23,8 @@
>  
>  # This test has only been verified with Linux targets, and would need
>  # to be generalized to support other targets
> -if {![istarget *-*-linux*]} {
> +
> +if {!([istarget *-*-linux*] || [istarget *-*-aix*])} {
>      return
>  }
>  

Could you update the comment to indicate that it's been verified to
work with (both) Linux and AIX targets?

Once that's done, you have my approval:

Approved-by: Kevin Buettner <kevinb@redhat.com>
diff mbox series

Patch

diff --git a/gdb/testsuite/gdb.threads/thread_events.exp b/gdb/testsuite/gdb.threads/thread_events.exp
index 100205e5f58..11a89139384 100644
--- a/gdb/testsuite/gdb.threads/thread_events.exp
+++ b/gdb/testsuite/gdb.threads/thread_events.exp
@@ -23,7 +23,8 @@ 
 
 # This test has only been verified with Linux targets, and would need
 # to be generalized to support other targets
-if {![istarget *-*-linux*]} {
+
+if {!([istarget *-*-linux*] || [istarget *-*-aix*])} {
     return
 }