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

Further fix issue #4376 posix platform #4387

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

mwichmann
Copy link
Collaborator

Oops, missed that SCons/Platform/virtualenvTests.py needed the same hack as SCons/Platform/PlatformTests.py (in PR #4381). THis is an additional test-only fix - does not require further updates to CHANGES as the comment there already covers it.

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

Oops, missed that SCons/Platform/virtualenvTests.py needed the same
hack as SCons/Platform/PlatformTests.py (in PR SCons#4381)

Signed-off-by: Mats Wichmann <[email protected]>
@mwichmann mwichmann added this to the NextRelease milestone Jul 27, 2023
@bdbaddog bdbaddog merged commit a7998e7 into SCons:master Jul 27, 2023
@mwichmann mwichmann deleted the test/posix-module-variant branch July 27, 2023 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants