Skip to content

Fix typos in CCFLAGS test#4692

Merged
bdbaddog merged 1 commit intoSCons:masterfrom
mwichmann:fix/ccflags-typo
Mar 6, 2025
Merged

Fix typos in CCFLAGS test#4692
bdbaddog merged 1 commit intoSCons:masterfrom
mwichmann:fix/ccflags-typo

Conversation

@mwichmann
Copy link
Collaborator

Trivial: in two DefaultEnvironment calls, tool=[] -> tools=[]. Doesn't affect the running of the tests, just enables the speedup as was intended.

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

tool=[] -> tools=[]

Signed-off-by: Mats Wichmann <mats@linux.com>
@mwichmann mwichmann added the testsuite Things that only affect the SCons testing. Do not use just because a PR has tests. label Mar 6, 2025
@bdbaddog bdbaddog merged commit cd43bf7 into SCons:master Mar 6, 2025
5 of 8 checks passed
@mwichmann mwichmann added this to 4.10 Mar 7, 2025
@mwichmann mwichmann moved this to Done in 4.10 Mar 7, 2025
@mwichmann mwichmann added this to the NextRelease milestone Mar 7, 2025
@mwichmann mwichmann deleted the fix/ccflags-typo branch March 7, 2025 15:55
@mwichmann mwichmann modified the milestones: NextRelease, 4.9.1 Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testsuite Things that only affect the SCons testing. Do not use just because a PR has tests.

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants