Hi Tom,

>Rather than a new #ifdef, could you just move the #include into the
>already-existing #ifdef block?  Thanks.

Thank you for noticing. I had missed it. I have made the change and sent a v2 of the patch requesting approval to commit.
Have a nice day ahead.

Regards,
Aditya.

From: Tom Tromey <tom@tromey.com>
Date: Monday, 13 January 2025 at 2:38 PM
To: Aditya Vidyadhar Kamath <akamath996@gmail.com>
Cc: Ulrich Weigand <Ulrich.Weigand@de.ibm.com>, tom@tromey.com <tom@tromey.com>, gdb-patches@sourceware.org <gdb-patches@sourceware.org>, Aditya Kamath <Aditya.Kamath1@ibm.com>, SANGAMESH MALLAYYA <sangamesh.swamy@in.ibm.com>
Subject: [EXTERNAL] Re: [PATCH] Fix AIX CI build break.

>>>>> Aditya Vidyadhar Kamath <akamath996@gmail.com> writes:

> A feature development made by the recent commit
> https://urldefense.proofpoint.com/v2/url?u=https-3A__sourceware.org_git_-3Fp-3Dbinutils-2Dgdb.git-3Ba-3Dcommitdiff-3Bh-3D6447969d0ac774b6dec0f95a0d3d27c27d158690&d=DwIBAg&c=BSDicqBQBDjDI9RkVyTcHQ&r=f-oUQ8ByG1nZ71OI9p76qywCPh7mxzU69hBYnkP9Nis&m=CiPaXJgRXT_qujr2bfmD3o2ZHPBYWNBHBALPUFfbCYLe1P7QtmWvyVaByGpz1-Ad&s=sAvE8OQri4yyxi5VGK59eE7PUDiFu6dffPlKHFMGn9U&e=
> missed to guard Python.h in HAVE_PYTHON macro.

> This commit is a fix for the same.

Sorry about that.

> +
> +#ifdef HAVE_PYTHON
>  #include "py-color.h"
> +#endif /* HAVE_PYTHON */

Rather than a new #ifdef, could you just move the #include into the
already-existing #ifdef block?  Thanks.

Tom