Skip to content

Separate UtilTests.py into individual files#4839

Merged
bdbaddog merged 2 commits intoSCons:masterfrom
Repiteo:util-test-split
Apr 12, 2026
Merged

Separate UtilTests.py into individual files#4839
bdbaddog merged 2 commits intoSCons:masterfrom
Repiteo:util-test-split

Conversation

@Repiteo
Copy link
Copy Markdown
Contributor

@Repiteo Repiteo commented Mar 20, 2026

This consolidates the Util test file separation into its own PR, as I'm no longer wholly convinced that the original PR is the right approach for "deprecating" imports.

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 RELEASE.txt (and read the README.rst).
  • I have updated the appropriate documentation

@mwichmann
Copy link
Copy Markdown
Collaborator

mwichmann commented Mar 21, 2026

Okay, that was weird stuff from one of the appveyor instances:

.\.appveyor\install.ps1
Python version check failed or Python not found at C:\Python312\python.exe. Installing Python312 via Chocolatey...
Chocolatey v0.10.15
Installing the following packages:
Python312
By installing you accept licenses for the packages.
python312 v3.12.10 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.
Chocolatey installed 0/1 packages. 
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Warnings:
 - python312 - python312 v3.12.10 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.
& : The term 'C:\Python312\python.exe' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try 
again.
At C:\projects\scons\.appveyor\install.ps1:22 char:20
+ $env:PYSITEDIR = & $pythonExe -c "import sys; print(sys.path[-1])"
+                    ~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Python312\python.exe:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Don't find 3.12, install 3.12 - but it's already installed, don't find 3.12?

@bdbaddog
Copy link
Copy Markdown
Contributor

Is this PR ready for review and merge?

@Repiteo
Copy link
Copy Markdown
Contributor Author

Repiteo commented Apr 12, 2026

Sorry, missed that message until now; this indeed was/is ready for testing! I can't comment on whatever appveyor weirdness occurred, though that's hopefully been resolved by whatever changes those files received recently

@bdbaddog bdbaddog merged commit 1f46738 into SCons:master Apr 12, 2026
8 of 12 checks passed
@Repiteo Repiteo deleted the util-test-split branch April 12, 2026 21:59
@mwichmann mwichmann added the maintenance Tasks to maintain internal SCons code/tools label Apr 13, 2026
@mwichmann mwichmann added this to the NextRelease milestone Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Tasks to maintain internal SCons code/tools

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants