Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: gdb-patches@sourceware.org
Subject: Add LIBDIR/gdb/python-<version> to Python search path
Date: Thu, 20 May 2010 18:40:00 -0000	[thread overview]
Message-ID: <1274380803-25266-1-git-send-email-brobecker@adacore.com> (raw)

Hello,

I wrote the following two patches in order to allow GDB's python
interpreter to import the modules provided by the python library
from a location that's relative to GDB itself.

The intent is to allow us to package up a GDB binary with python support.
Since we do not know whether the target system on which GDB is going to
be installed has the correct version of python (or even if it has, whether
it has been built correctly), we want to also package up the python
library as well.  That way, regardless of what the target system provides,
we know the debugger will be using a well-defined version of the python
library.

The location I chose for that is GDB_LIBDIR/python-<version>, where
GDB_LIBDIR is LIBDIR/gdb by default.  So, for instance, with a prefix
set to /my/prefix/dir, a default GDB_LIDIR, and Python version 2.5,
we'd be using: /my/prefix/dir/lib/gdb/python-2.5.

So the first patch introduces --with-gdb-libdir as a new command-line
configure switch.  This allows us to change GDB_LIBDIR to something
different.

The second patch makes use of this new variable by adding the python
lib dir at the start of the python search path.  We can possibly
make it more configurable for the user by allowing him to override
the path to the python library (Eg: --with-gdb-libpythondir), but
I don't think that this is going to bring much.

Any thoughts on the approach? Obviously, this also needs documentation,
but I'd rather have the go-ahead before I start documenting the new
switch...

Thanks,
-- 
Joel


             reply	other threads:[~2010-05-20 18:40 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-20 18:40 Joel Brobecker [this message]
2010-05-20 18:40 ` [PATCH 2/2] Add gdb_libdir/python-<version> in front of " Joel Brobecker
2010-05-20 21:37   ` Jan Kratochvil
2010-05-20 21:49     ` Joel Brobecker
2010-05-20 22:09       ` Jan Kratochvil
2010-05-20 23:03         ` Joel Brobecker
2010-05-21  9:33   ` Doug Evans
2010-05-21 16:01     ` Joel Brobecker
2010-05-21 16:17     ` Tom Tromey
2010-05-21 19:39       ` Doug Evans
2010-05-20 18:54 ` [PATCH 1/2] Add support for --with-gdb-libdir (default: <prefix>/lib/gdb) Joel Brobecker
2010-05-27 15:04   ` Joel Brobecker
2010-05-28  5:27     ` Doug Evans
2010-05-28 16:08       ` Joel Brobecker
2010-05-20 19:03 ` Add LIBDIR/gdb/python-<version> to Python search path Daniel Jacobowitz
2010-05-20 19:15   ` Joel Brobecker
2010-05-20 19:23     ` Daniel Jacobowitz

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=1274380803-25266-1-git-send-email-brobecker@adacore.com \
    --to=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    /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