Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: David Edelsohn <dje.gcc@gmail.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: Andreas Schwab <schwab@suse.de>,
	"gdb@sourceware.org" <gdb@sourceware.org>
Subject: Re: GDB Python extension on AIX
Date: Fri, 19 Aug 2016 15:36:00 -0000	[thread overview]
Message-ID: <CAGWvny=hQZD+dEo3HCisM7m7A5ajoa1ne_pGEXQ424YU0cO1sQ@mail.gmail.com> (raw)
In-Reply-To: <20160819054928.GB2983@adacore.com>

On Fri, Aug 19, 2016 at 1:49 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> I'm not sure what a "standard" location is on AIX.  I was using the
>> Bull Freeware installation of Python in /opt/freeware.
>>
>> Was the Python installation found and recognized on your AIX system?
>> Until some of my more recent changes, GDB configure did not consider
>> the installation correct to try to build the GDB feature.
>
> In our case, we configure GDB with --with-python=/path/to/python,
> so it doesn't need to be a location that the configury would find
> by itself. In your case, I think what's happening is that you have
> /opt/freeware/bin in your PATH, and that allows the GDB configury
> to find that python and therefore use it.

This probably relates to the original conflict between implicit and
explicit paths that you were trying to fix

Would the appended patch be an acceptable approach?

Thanks, David

gdb/
* python/python-internal.h: Don't redefine _POSIX_C_SOURCE and
_XOPEN_SOURCE on AIX.

diff --git a/gdb/python/python-internal.h b/gdb/python/python-internal.h
index 8545c7b..6378ccc 100644
--- a/gdb/python/python-internal.h
+++ b/gdb/python/python-internal.h
@@ -72,8 +72,12 @@
    To work around this, undef _POSIX_C_SOURCE before we include Python.h.

    Same problem with _XOPEN_SOURCE.  */
+
+/* A kludge to avoid overriding on AIX.  */
+#ifndef _AIX
 #undef _POSIX_C_SOURCE
 #undef _XOPEN_SOURCE
+#endif

 /* On sparc-solaris, /usr/include/sys/feature_tests.h defines
    _FILE_OFFSET_BITS, which pyconfig.h also defines.  Same work


  reply	other threads:[~2016-08-19 15:36 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-11  1:56 David Edelsohn
     [not found] ` <mvmk2fnsr0a.fsf@hawking.suse.de>
2016-08-11 13:56   ` David Edelsohn
2016-08-18  7:21     ` Joel Brobecker
2016-08-18 13:00       ` David Edelsohn
2016-08-19  5:49         ` Joel Brobecker
2016-08-19 15:36           ` David Edelsohn [this message]
2016-08-31 23:41             ` Joel Brobecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAGWvny=hQZD+dEo3HCisM7m7A5ajoa1ne_pGEXQ424YU0cO1sQ@mail.gmail.com' \
    --to=dje.gcc@gmail.com \
    --cc=brobecker@adacore.com \
    --cc=gdb@sourceware.org \
    --cc=schwab@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox