Skip to content

Add workflow to verify CMakeLists.txt files - #896

Merged
natecook1000 merged 1 commit into
mainfrom
verify-runtimes-cmakelists
Oct 8, 2026
Merged

natecook1000 merged 1 commit into
mainfrom
verify-runtimes-cmakelists

Conversation

@natecook1000

@natecook1000 natecook1000 commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

The _StringProcessing and _RegexParser modules are built as a part of the standard library using CMakeLists.txt files in the swiftlang/swift repository, so files added in this project will break that build if the configuration is not updated in concert. This new workflow should help flag that issue before merging - it both verifies that the local versions of the CMake configuration files are in sync with their swiftlang/swift counterparts and completes a CMake build to test that the configuration is up to date with the libraries' current code.

Note that for changes that require a CMakeLists.txt update, this job will still error — cross-repository testing from the Swift project should be used to validate that things are actually set up properly.

@natecook1000
natecook1000 force-pushed the verify-runtimes-cmakelists branch from 88972be to 522a2ec Compare October 8, 2026 15:46
The _StringProcessing and _RegexParser modules are built as a part of the
standard library using CMakeLists.txt files in the swiftlang/swift
repository, so files added in this project will break that build if the
configuration is not updated in concert. This new workflow should help
flag that issue before merging - it both verifies that the local versions
of the CMake configuration files are in sync with their swiftlang/swift
counterparts and completes a CMake build to test that the configuration is
up to date with the libraries' current code.

Note that for changes that require a CMakeLists.txt update, this job
will still error - cross-repository testing from the Swift project should
be used to validate that things are actually set up properly.
@natecook1000
natecook1000 force-pushed the verify-runtimes-cmakelists branch from 522a2ec to e6a9011 Compare October 8, 2026 17:02
@natecook1000
natecook1000 merged commit 0bec31a into main Oct 8, 2026
26 checks passed
@natecook1000
natecook1000 deleted the verify-runtimes-cmakelists branch October 8, 2026 18:46
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.

1 participant