Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PyPackageDir #4479

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Fix PyPackageDir #4479

merged 1 commit into from
Feb 14, 2024

Conversation

mwichmann
Copy link
Collaborator

Passing a module to PyPackageDir which the active Python (system or virtualenv) cannot locate through the import machinery would cause SCons to fail with an AttributError, because the result of the initial lookup was used without checking for success. Now returns None for not found. Manpage entry and docstring also updated.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt (and read the README.rst)
  • I have updated the appropriate documentation

Passing a module which the active Python (system or virtualenv)
cannot locate through the import machinery would cause SCons to fail
with an AttributError, because the result of the initial lookup was
used without checking for success. Now returns None for not found.
Manpage entry and docstring also updated.

Signed-off-by: Mats Wichmann <[email protected]>
@bdbaddog bdbaddog merged commit b8fffb3 into SCons:master Feb 14, 2024
4 of 6 checks passed
@mwichmann mwichmann added this to the 4.7 milestone Feb 14, 2024
@mwichmann mwichmann deleted the doc/PyPackageDir branch February 14, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants