From: Doug Evans <dje@google.com>
To: tromey@redhat.com
Cc: Joel Brobecker <brobecker@adacore.com>, gdb-patches@sourceware.org
Subject: Re: [RFA/python:1/2] Add support for --with-pythondir.
Date: Fri, 28 May 2010 09:55:00 -0000 [thread overview]
Message-ID: <AANLkTimLm7I7k5J-emVPuysb0qgyiq8WOXfpUhylTSb8@mail.gmail.com> (raw)
In-Reply-To: <m3eigxf6hf.fsf@fleche.redhat.com>
On Thu, May 27, 2010 at 1:13 PM, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> "Joel" == Joel Brobecker <brobecker@adacore.com> writes:
>
> Joel> 2010-05-26 Tom Tromey <tromey@adacore.com>
> Joel> Joel Brobecker <brobecker@adacore.com>
> Joel> * configure.ac: Add handling of --with-pythondir.
> Joel> * configure, config.in: Regenerate.
> Joel> * python/python.c (_initialize_python): Add variable "pythondir"
> Joel> inside module gdb. Initialize variable gdb.__path__ and execute
> Joel> __init__.py if found in subdirectory "gdb" of the pythondir.
>
> FWIW I read it and it seems ok to me.
> Thanks for doing this.
fwiw, I still have some concerns.
Support doesn't have to be included in the patch, but IWBN to have a
plan for how to support .so's associated with gdb python modules in a
way that follows Python's scheme: i.e. .py*'s go in
$prefix/lib/pythonX.Y and .so's go in
$exec_prefix/lib/pythonX.Y/lib-dynload [i.e. architecture independent
files go in $prefix and architecture dependent files go in
$exec_prefix].
If the plan is to add --with-python-sodir [or some such] if/when it's
needed, great. Or if the plan is to decide to just not support it,
that's probably ok too. I'd just like to put in the thought now.
Also, do we want to name this --with-gdb-pythondir? [and do we want to
rename --with-python to --with-gdb-python?]
The idea being that if gdb was ever configured with something else
that had its own with for --with-python*, then gdb's wouldn't collide.
I don't know if it's that important, just thought I'd bring it up.
Apologies if it's already been discussed and decided and I've
forgotten.
next prev parent reply other threads:[~2010-05-28 5:27 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-27 0:09 Add support for the GDB Python Library Joel Brobecker
2010-05-27 0:09 ` [RFA/python:2/2] First script in GDB python library - command/pahole.py Joel Brobecker
2010-05-27 20:37 ` Tom Tromey
2010-05-28 1:34 ` Joel Brobecker
2010-06-03 0:11 ` Joel Brobecker
2010-06-03 6:28 ` Doug Evans
2010-06-03 15:35 ` Joel Brobecker
2010-07-11 18:49 ` Tom Tromey
2010-05-27 0:25 ` [RFA/python:1/2] Add support for --with-pythondir Joel Brobecker
2010-05-27 6:49 ` Doug Evans
2010-05-27 20:32 ` Tom Tromey
2010-05-28 9:55 ` Doug Evans [this message]
2010-05-28 17:20 ` Joel Brobecker
2010-05-30 17:10 ` Doug Evans
2010-06-01 20:15 ` Joel Brobecker
2010-06-01 20:39 ` Doug Evans
2010-06-01 20:53 ` Joel Brobecker
2010-05-27 1:54 ` Add support for the GDB Python Library Doug Evans
2010-05-27 3:42 ` Doug Evans
2010-05-27 15:17 ` 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=AANLkTimLm7I7k5J-emVPuysb0qgyiq8WOXfpUhylTSb8@mail.gmail.com \
--to=dje@google.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
--cc=tromey@redhat.com \
/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