pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/python/cpython/pull/151288

fc5db.css" /> gh-151096: Fix test_embed with split exec prefix by harjothkhara · Pull Request #151288 · python/cpython · GitHub
Skip to content

gh-151096: Fix test_embed with split exec prefix#151288

Open
harjothkhara wants to merge 1 commit into
python:mainfrom
harjothkhara:gh-151096-test-embed-split-prefix
Open

gh-151096: Fix test_embed with split exec prefix#151288
harjothkhara wants to merge 1 commit into
python:mainfrom
harjothkhara:gh-151096-test-embed-split-prefix

Conversation

@harjothkhara

@harjothkhara harjothkhara commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Fix test_init_is_python_build_with_home for builds configured with different --prefix and --exec-prefix.

When running from the build directory on POSIX, getpath uses PREFIX for prefix/base_prefix and EXEC_PREFIX for exec_prefix/base_exec_prefix. The test expected both values to match sys.prefix, which fails for split-prefix builds.

This also updates module_search_paths() so its default exec_prefix comes from the expected config’s exec_prefix instead of prefix.

Validation:

  • Configured a local split-prefix build with --prefix=/private/tmp/cpython-151096-install --exec-prefix=/private/tmp/cpython-151096-install/exec.
  • Before the fix, ./python.exe -m test test_embed -v -m test_init_is_python_build_with_home failed with mismatched exec_prefix / base_exec_prefix.
  • After the fix, ./python.exe -m test test_embed -v -m test_init_is_python_build_with_home passes.
  • Full ./python.exe -m test test_embed -v passes.

This is a tests-only change; skip news.

@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Jun 10, 2026
@harjothkhara

Copy link
Copy Markdown
Contributor Author

This is a tests-only change. Could a triager please add the skip news label?

@harjothkhara harjothkhara marked this pull request as ready for review June 10, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy