Short-term getpath fix

[geofft]

TODO description, just want to get CI running

[jjhelmus]

Passing the path the the executable via the library variable in getpath.py has a smell as the variable is described as being the path to the dylib/DLL/so. When static linking there is no shared library so I see the logic in passing the path to the binary (as it is a DSO) itself.

[jjhelmus]

With this patch the directory pointed to by home in the pyvenv.cfg is still searched if the library search fails (but it should not fail).

The change from python/cpython#106045 (comment) (below) would update this path to the directory of the base_executable when it exists. I'm somewhat partial to that solution in addition to this change.

@@ -411,6 +412,9 @@ def search_up(prefix, *landmarks, test=isfile):
                             if isfile(candidate):
                                 base_executable = candidate
                                 break
+                if base_executable and isfile(base_executable):
+                    # Update the executable directory to be based on the resolved base executable
+                    executable_dir = real_executable_dir = dirname(base_executable)
             # home key found; stop iterating over lines
             break

[jjhelmus]

I've found following helpful check/reproduce/debug this issue:

./configure --prefix=/example
make -j
make DESTDIR=<some_real_path> install
cd <some_real_path>
./example/bin/python3.14 # can be used to create symlinks/venv/etc 

[zanieb]

Did you mean to close this?

[jjhelmus]

Did you mean to close this?

Nope, oops