Skip to content

Update Fortran FLAGS tests#4710

Merged
bdbaddog merged 2 commits intoSCons:masterfrom
mwichmann:test/fort-flags-live
Apr 11, 2025
Merged

Update Fortran FLAGS tests#4710
bdbaddog merged 2 commits intoSCons:masterfrom
mwichmann:test/fort-flags-live

Conversation

@mwichmann
Copy link
Collaborator

@mwichmann mwichmann commented Apr 10, 2025

Tests formerly containing both are split into a flags-handling-only section using a mocked tool, and a live section using a real compiler, in a new file. General cleanup. Added shared-object-flags tests for two dialects that didn't have one.

Test-only changes, so no doc impacts.

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

Tests contain both are split into a flags-handling-only section using
a mocked tool, and a live section using a real compiler, in a new file.
General cleanup. Added shared-object-flags tests for two dialects that
didn't have one.

Signed-off-by: Mats Wichmann <mats@linux.com>
@mwichmann mwichmann added testsuite Things that only affect the SCons testing. Do not use just because a PR has tests. Fortran Fortran support issues labels Apr 10, 2025
@mwichmann mwichmann added this to 4.10 Apr 10, 2025
@mwichmann mwichmann moved this to In review in 4.10 Apr 10, 2025
@bdbaddog bdbaddog merged commit aafb3a6 into SCons:master Apr 11, 2025
5 of 8 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in 4.10 Apr 11, 2025
@mwichmann mwichmann added this to the NextRelease milestone Apr 12, 2025
@mwichmann mwichmann deleted the test/fort-flags-live branch April 12, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fortran Fortran support issues 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