Fixed GH Issue 4809. scoop.sh installed scons (and likely scons-local )created msvs project files would fail. #4817
+81
−66
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…) created msvs project files would fail.
This was referenced Jan 18, 2026
jcbrill
added a commit
to jcbrill/scons
that referenced
this pull request
Jan 20, 2026
Changes: * Change SCons/Tool/msvs.py function comment before getExecScriptMain and replace with docstring. Taken from PR SCons#4817. * Revert python executable code * Generate scons_home (user env, SCONS_HOME, or SCONS_LIB_DIR path specification) and scons_path (currently executing SCons module path) paths in unified script in SCons/Tool/msvs.py and testing/framework/TestSConsMSVS.py. Similar to the two code paths in PR SCons#4817. * Set the SCONS_HOME variable in the testenv as is done with SCONS_LIB_DIR. Add SCONS_HOME to testing/framework/TestSConsMSVS.py.
jcbrill
reviewed
Jan 20, 2026
…e scons.py which was run available in SCons.Script.Main.SCONS_SCRIPT_PATH
jcbrill
reviewed
Jan 26, 2026
|
|
||
|
|
||
| def main() -> None: | ||
| def main(script_path) -> None: |
Contributor
There was a problem hiding this comment.
Bugs of omission?
SCons/SCript/Main main function now takes a required script_path argument.
Missing:
SCons/__main__.pyneeds to pass ascript_pathargument.- The
bin/scons-test.pyfunctiondo_nothingneeds to accept ascript_pathargument.
I could be wrong.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed GH Issue #4809 4809. scoop.sh installed scons (and likely scons-local )created msvs project files would fail.
See #4810 for discussion.
This is the implementation of the fix discussed in that PR.
Largely the work of.
@jcbrill & @madmiraal
Note: I tested this live with the SConstruct from #4809 and scoop.sh installed scons
(And then locally editing scoop installed scons' msvs.py"
Contributor Checklist:
CHANGES.txtandRELEASE.txt(and read theREADME.rst).