From: Andrew Burgess <andrew.burgess@embecosm.com>
To: Simon Marchi <simark@simark.ca>
Cc: gdb-patches@sourceware.org, Richard Bunt <richard.bunt@arm.com>
Subject: Re: [PATCH] gdb: Allow more control over where to find python libraries
Date: Sat, 08 Feb 2020 00:22:00 -0000 [thread overview]
Message-ID: <20200208002226.GK4020@embecosm.com> (raw)
In-Reply-To: <7cbcb092-3a66-32d5-f09d-2a90443a1cc4@simark.ca>
* Simon Marchi <simark@simark.ca> [2020-02-07 12:58:46 -0500]:
> On 2020-02-06 11:46 a.m., Andrew Burgess wrote:
> > The motivation behind this commit is to make it easier to bundle the
> > Python libraries with GDB when linking GDB against a static
> > libpython, the Python libraries will be manually added into the GDB
> > installation tree, and GDB should be able to find them at run-time.
> > The installation tree will look like this:
> >
> > .
> > |-- bin/
> > |-- include/
> > |-- lib/
> > | `-- python3.8/
> > `-- share/
> >
> > The benefit here is that the entire installation tree can be bundled
> > into a single archive and copied to another machine with a different
> > version of Python installed, and GDB will still work, including its
> > Python support.
>
> For those who might be wondering, like me: isn't the goal of linking
> statically with the lib to avoid having an external library? This
> patch actually deals with finding the .py files provided by the Python
> standard library, not the native code. The native code is indeed
> linked statically inside the gdb executable.
Yes, this is it exactly. I will rewrite the commit message to say '.py
files' instead as that's much clearer.
Thanks,
Andrew
>
> I won't pretend to understand in details what's happening with the Python
> detection in configure (I find it's quite messy), but I didn't spot anything
> wrong with your patch, and it seems to address a valid use case, so I'm not
> against it.
>
> Simon
next prev parent reply other threads:[~2020-02-08 0:22 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-06 16:46 Andrew Burgess
2020-02-06 18:31 ` Eli Zaretskii
2020-02-19 15:53 ` Andrew Burgess
2020-02-19 15:57 ` Eli Zaretskii
2020-02-07 17:58 ` Simon Marchi
2020-02-08 0:22 ` Andrew Burgess [this message]
2020-02-19 16:27 ` [PATCHv2] " Andrew Burgess
2020-02-19 17:09 ` Eli Zaretskii
2020-02-20 17:00 ` Tom Tromey
2020-02-20 19:22 ` Andrew Burgess
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=20200208002226.GK4020@embecosm.com \
--to=andrew.burgess@embecosm.com \
--cc=gdb-patches@sourceware.org \
--cc=richard.bunt@arm.com \
--cc=simark@simark.ca \
/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