Issue18309
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2013-06-26 16:00 by mathias, last changed 2022-04-11 14:57 by admin.
Files | ||||
---|---|---|---|---|
File name | Uploaded | Description | Edit | |
python-relative-path-lookup.diff | mathias, 2013-06-26 16:00 | Proposed change | review | |
python-relative-path-lookup-v2.diff | mathias, 2013-07-04 07:56 | review | ||
workaround.c | shakfu, 2022-03-15 05:34 | example of workaround to obtain python home in a relocatable plugin scenario. |
Messages (17) | |||
---|---|---|---|
msg191909 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2013-06-26 16:00 | |
Hi all, I want to move python a bit closer to be relocatable. One problem to solve is where python finds its modules. The usual lookup mechanism is to compile in a configure time determined prefix that is used as a last resort path if the paths are not set otherwise during application/interpreter startup. The most commonly known way to change the module path at startup time are probably the environment variables PYTHONPATH and PYTHONHOME. The python interpreter itself already tries to interpret argv[0] to get to this point, but it would be nice if an application embedded interpreter also finds its module path without providing this argv[0] directly to the python library. This should even work if being moved or being installed at a different path than the configure time prefix path. The proposal is to add an additional attempt to find the python modules just before we resort to the compiled in prefix by looking at the path to the python27.{so,dll}. Relative to this shared object python library file the python modules are searched in the usual way. If there are no python modules found relative to the python library file, the very last resort compiled in prefix is used as usual. For architectures where we cannot determine the path of the shared library file, nothing changes. I have attached a patch that tries to implement this. It should serve as a base for discussions. This change is tested on linux and behaves like expected. The windows code for this is copied over from an other project where I have this actively running. But this python code variant is not even compile tested on windows. thanks in advance Mathias |
|||
msg191929 - (view) | Author: Eric Snow (eric.snow) * | Date: 2013-06-27 04:34 | |
Hi Mathias. There is a current proposal (http://www.python.org/dev/peps/pep-0432/) for improving interpreter startup. So changes in this area are subject to extra caution. The changes you are talking about are at least indirectly impacted by the proposal, though I expect they are more directly tied to what happens in site.py. As to your proposal, aren't the embedding needs already addressed? See http://docs.python.org/2/c-api/intro.html#embedding-python. Is there some convention for keeping the site files adjacent to the SO/DLL that would warrant your proposed code? p.s. this would be a new feature so it only applies to Python 3.4. |
|||
msg191935 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2013-06-27 09:17 | |
Hi Eric, Thanks for looking at that ticket so fast! Reassigning this to 3.4 is great. In general, yes I can already do what I need more or less. This is the reason why I can be fine with about every python version. The point I bring up this change that I believe I am doing this at an unappropriate place as I need to know some internals of python when I do so and that I think that other can probably also benefit from this idea/change. What I currently do is to write an application that just uses python*.so as an embedded interpreter and this precompiled application might be relocated to about everywhere - just where it is unpacked. We are currently using the same sort of code to find out where the python*so file is and we use Py_SetPythonHome to set is to the directory where the so file resides. Why are we doing this? So, it takes the idea that is currently in the standard python interpreter. This one tries to be relocatable (means: pack the installation directory and unpack that somewhere else and be still able to run) by looking at argv[0] and dereferencing symbolic links until it arrives at a real file. Now suppose you want to embed python, then you do no longer use the standard python interpreter program. You may also use a different installation layout for basic things like bin and lib. So you end up with an application that is no longer able to find its provided python modules by looking at the applications path. But instead of starting from the path of the interpreter (which is not used in this case) or the application itself you could start from the python library path and look for your python installation relative to that. So as long as you stick with the relative file layout of everything that is python related (and only what is python related, the python.so and the modules) when you pack and unpack your precompiled application this would just work. So, put that in short: Instead of dynamically finding the the python module path relative to .../bin/python try to find the python relative to .../lib/libpython34.so. The benefit of that would be that every application that embeds python and needs to be relocatable will just work in the way that today only the standard python interpreter works. I try to get all of the PEP you pointed me to. As I am seeing this longer document the first time, I am not sure if I missed something there, but in that framework of this my proposal would probably influence the initial setting of sys.prefix (?) if this is not already provided from the embedding application. And yes I am perfectly fine with a different or more general approach. The initially attached patch is something that tried to integrate into the current checked in code as I understood it. Greetings Mathias |
|||
msg191944 - (view) | Author: Alyssa Coghlan (ncoghlan) * | Date: 2013-06-27 11:42 | |
The way we figure out where to find the standard library is crazy, and creating the infrastructure to start making it less crazy is actually one of the prime motivations for PEP 432 :) |
|||
msg191994 - (view) | Author: Ronald Oussoren (ronaldoussoren) * | Date: 2013-06-28 14:52 | |
Note that the OSX port already does this for framework builds. I don't know why we don't use the same code for shared library builds. Issue #15498 contains a patch that switches this code from a deprecated nextstep-era API to dladdr. Two comments on the patch attached to this issue: 1) The name "_PyImport_GetModulePath" is confusing, I'd use _PyImport_GetSharedLibPath to make clear that this is locating the shared library. 2) The code calls dladdr on a static variable that's introduced just for that, it is also possible to call dladdr on an already existing symbol (for example the address of a function in the public API). |
|||
msg192277 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2013-07-04 07:56 | |
Hi Ronald, Eric, Nick, Looking up the symbol name of the current function should work also. And I am free to rename these functions to whatever you like. Attached is version 2 of the patch with the suggested changes. The windows implementation is still untested. It would be interesting to know if this kind of lookup scheme can be included into PEP 432. Provided the spirit of this PEP, I can imagine to provide several functions to build up the pythonpath starting from something. So say, have a 'get python path from argv[0]', a 'get python path from shared python library' and a 'get python path from prefix' function (I may miss a variant). Also a 'build python path from python home root entry point' function would be useful and could be used by the above functions. An application embedding python can then call those functions that are sensible for its own use and installation scheme to set the module path in the PyConfig struct. The Py_ReadConfig function will internally use the above suggested functions to build up the default configuration if not already provided. Greetings Mathias |
|||
msg353243 - (view) | Author: STINNER Victor (vstinner) * | Date: 2019-09-26 00:47 | |
The PEP 587 "Python Initialization Configuration" has been implemented in Python 3.8. It provides fine control on the "Path Configuration": * https://docs.python.org/dev/c-api/init_config.html * https://docs.python.org/dev/c-api/init_config.html#init-path-config |
|||
msg353267 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2019-09-26 06:31 | |
Hi, Nice to see some progress. Still, I checked todays https://github.com/python/cpython.git master and 3.8 branch (is that the current cpython development code?). Neither of them contain a call to dladdr beside the macos code path mentioned in msg191994 by Ronald Oussoren which does this already for a long time. By the lack of dladdr, I conclude that the code idea of my request here is not solved. May be to rephrase that. The basic idea behind that request was to make pythons default way to setup the paths required to find the python modules based on the place where the python library resides instead of the python executable program. I do not mean the compile time prefix but the actual location of the shared object in the file system. That would help to build applications that embed cpython, ship and unpack the whole application tree including the python modules to a custom location, while still preserving the subtree structure containing the python shared library and the python modules, not known at compile time. Note that this patch contained code to make that work from within python without custom code in the embedding application. Doing that on the embedding and calling application side was always possible and still is possible - but that was not the point. best Mathias |
|||
msg353276 - (view) | Author: STINNER Victor (vstinner) * | Date: 2019-09-26 07:29 | |
Hum, I am confused. I understood that this issue is able customizing sys.path when Python is embedded. But it seems like the feature request is more about the *default* implementation, not how to reimplement it outside Python (with custm code). |
|||
msg353288 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2019-09-26 10:11 | |
Yes. msg191944 from Nick Coghlan, made me think that with all the initialization rework that appeared to be underway you want to incorporate that presented idea of basing the default onto the location of the libpython.so or the pythonX.X.dll instead of the location of python/python.exe. And as mentioned by Ronald Oussoren that would even align the methods used across the architectures to something common with a fallback to the current way that takes the path of the interpreter executable. At least that is what the provided patch implemented in the old code structure. And this does not even change the default for the common case where the default is plain useful. It is just changing the way how the default is determined so that the default for the case of an embedded interpreter is more meaningful. As stated somewhere above. The you can do that with application code when setting up the embedded interpreter, but it would be nice if that just works out of the box and by that helps applications not thinking of that solution. best Mathias |
|||
msg353311 - (view) | Author: STINNER Victor (vstinner) * | Date: 2019-09-26 14:47 | |
My plan is not to change the default implementation to calculate the path configuration, but make it easier to customize the path configuration. One idea is to rewrite Modules/getpath.c and PC/getpathp.c in Python and convert it to a frozen module. It is easier to modify Python code than C code. In the past, we already did such change for importlib (which also has a frozen part, importlib._bootstrap and importlib._bootstrap_external). The PEP 587 implementation moves towards that with the "Multi-Phase Initialization Private Provisional API": https://docs.python.org/dev/c-api/init_config.html#multi-phase-initialization-private-provisional-api |
|||
msg353720 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2019-10-02 05:47 | |
Ok, so far. But what shall I do now? It would be nice that python is a bit smarter in finding its increasing important module files when being embedded into an application. Anybody out there who wants to look at that contribution? best Mathias |
|||
msg415142 - (view) | Author: Shakeeb Alireza (shakfu) | Date: 2022-03-14 14:22 | |
I have exactly the same need and use-case as Mathias in my project which includes a requirement to embed python3 in a relocatable folder structure w which serves as an application package (https://github.com/shakfu/py-js). This can be done using the Framework structure, thanks to Greg Neagle's solution referenced in (https://bugs.python.org/issue42514), but not for any python builds with --enabled-shared. In any case, providing options (at the c-level or otherwise, for embedded applications as described by Mathias would be ideal: "I can imagine to provide several functions to build up the pythonpath starting from something. So say, have a 'get python path from argv[0]', a 'get python path from shared python library' and a 'get python path from prefix' function (I may miss a variant)." |
|||
msg415145 - (view) | Author: STINNER Victor (vstinner) * | Date: 2022-03-14 14:47 | |
In Python 3.11, Modules/getpath.c has been rewritten in Python: Modules/getpath.py. Maybe it's now simpler to hack this file. But you must rebuild Python to take changes in account. |
|||
msg415148 - (view) | Author: Shakeeb Alireza (shakfu) | Date: 2022-03-14 15:23 | |
Thanks, Victor. I can imagine getpath.py will be more hackable (even if it is frozen). Still, it replicates the old algorithm: # Before any searches are done, the location of the executable is # determined. If Py_SetPath() was called, or if we are running on # Windows, the 'real_executable' path is used (if known). Otherwise, # we use the config-specified program name or default to argv[0]. In my case (and I think for Mathias), the executable is a non python application and what is actually dynamically linking to libpythonX.Y.dylib (built via --enable-shared) is a c-based plugin (which calls PyInitialize()), and these two are only aware of their relative locations via the @rpath, @loader_path mechanism. Currently, in this scenario, libpythonX.Y.dylib doesn't know here pythonhome is unless explicitly told via PySetPath() or it defaults to the hardcoded sys.prefix . If this is to be relocatable, then PySetPath() should be able to handle relative paths. |
|||
msg415172 - (view) | Author: Mathias Fröhlich (mathias) | Date: 2022-03-14 17:46 | |
Hey, Shakeeb Alireza is right, the original problem was an application that links and embeds against libpython*{so,dll,dynlib} and should more easily find components like /*/lib*/python*.*/site.py and most probably now it needs to find getpath.py as well. While I am no longer working on that application where I wanted to have that feature, I still believe it would be worthwhile to find the *.py files in the file system relative to the location of the libpython*{so,dll,dynlib} file. Thanks for taking care. Mathias |
|||
msg415218 - (view) | Author: Shakeeb Alireza (shakfu) | Date: 2022-03-15 05:34 | |
Thanks, Mathias. This is all about improving python's 'relocatability'. Just to expand on my prior point: the scenario we are talking about is where one embeds python in a host application's plugin system rather than in the host application itself. In this case, sys.executable is the host application and a relocatable plugin embeds a 'python client'. If a full python distribution is not bundled within this client[*], it needs to (1) link to libpythonX.Y.dylib and (2) get the location of the standard library. There are standard cross-platform methods for (1) to be achieved by way of symmetrical @rpath lookups on the client and libpythonX.Y.dylib sides. So this resolvable even in the case when python is compiled with --enabled-shared. However, even if (1) is achieved, the client cannot get, programmatically via the python c-api, the location of libpythonX.Y.dylib (even if it is properly dynamically linking to it), because it cannot rely on sys.executable. I think this is crux of Mathias' argument. Of course there are workarounds, but they are (at least to me) all platform specific. The first and easiest is to just build using the Framework structure and don't ever use --enable-shared, provided you find Greg Neagle's solution (https://bugs.python.org/issue42514) Another workaround which is specific to my context (which I have attached), is to use Apple's CoreFoundation library to get the path to the plugin bundle and from there find our way to the python distribution in the containing folder structure (package). [*] It is possible to insert a full python distribution into a bundle (in the osx meaning), but then it becomes necessarily frozen or 'sealed' due to Apple's codesigning and notarization requirements, and it basically means that the user cannot extend it with further installations of python packages which contain c-extensions unless they jump through some additional codesigning and notarization hoops. |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:57:47 | admin | set | github: 62509 |
2022-03-18 10:35:06 | vstinner | set | nosy:
- vstinner |
2022-03-15 05:34:35 | shakfu | set | files:
+ workaround.c messages: + msg415218 |
2022-03-14 17:46:33 | mathias | set | messages: + msg415172 |
2022-03-14 15:23:11 | shakfu | set | messages: + msg415148 |
2022-03-14 14:47:14 | vstinner | set | messages: + msg415145 |
2022-03-14 14:22:50 | shakfu | set | nosy:
+ shakfu messages: + msg415142 |
2019-10-02 05:47:38 | mathias | set | messages: + msg353720 |
2019-09-26 14:47:00 | vstinner | set | messages: + msg353311 |
2019-09-26 10:11:35 | mathias | set | messages: + msg353288 |
2019-09-26 07:29:21 | vstinner | set | messages:
+ msg353276 versions: + Python 3.9, - Python 3.8 |
2019-09-26 06:31:15 | mathias | set | status: closed -> open resolution: fixed -> messages: + msg353267 |
2019-09-26 00:47:29 | vstinner | set | status: open -> closed versions: + Python 3.8, - Python 3.4 nosy: + vstinner messages: + msg353243 resolution: fixed stage: resolved |
2013-07-04 07:56:08 | mathias | set | files:
+ python-relative-path-lookup-v2.diff messages: + msg192277 |
2013-06-28 14:52:47 | ronaldoussoren | set | nosy:
+ ronaldoussoren messages: + msg191994 |
2013-06-27 11:42:00 | ncoghlan | set | messages: + msg191944 |
2013-06-27 09:17:32 | mathias | set | messages: + msg191935 |
2013-06-27 04:34:44 | eric.snow | set | versions:
- Python 2.6, Python 3.1, Python 2.7, Python 3.2, Python 3.3, Python 3.5 nosy: + ncoghlan, eric.snow messages: + msg191929 components: + Interpreter Core, - Installation |
2013-06-26 16:00:49 | mathias | create |