Skip to content

Conversation

Lukasa
Copy link
Contributor

@Lukasa Lukasa commented Jul 22, 2025

The test runners for Windows were failing to load the paths from disk for test vectors. This patch cleans up the loading logic.

(cherry picked from commit 1f483f4)

The test runners for Windows were failing to load the paths from disk
for test vectors. This patch cleans up the loading logic.

(cherry picked from commit 1f483f4)
Copy link
Member

@etcwilde etcwilde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@Lukasa Lukasa added the 🔨 semver/patch No public API change. label Jul 22, 2025
Copy link
Member

@fabianfett fabianfett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Lukasa Lukasa merged commit 785709b into apple:3.12.x Jul 22, 2025
29 of 30 checks passed
@Lukasa Lukasa deleted the cb-backport-350-to-3.12 branch July 22, 2025 18:01
Steelskin added a commit to Steelskin/swift that referenced this pull request Jul 23, 2025
Changelog:
* apple/swift-cyrpto@d79c573 Make Build: Disable CMP0157 (apple/swift-crpto#387)
* apple/swift-cyrpto@ebcbfc6 build: Make swift-crypto build on Windows (apple/swift-crypto#385)
* apple/swift-crypto@785709b Fix broken Windows tests (apple/swift-crypto#350) (apple/swift-crypto#386)
Steelskin added a commit to Steelskin/swift that referenced this pull request Jul 23, 2025
Changelog:
* apple/swift-crypto@d79c573 Make Build: Disable CMP0157 (apple/swift-crpto#387)
* apple/swift-crypto@ebcbfc6 build: Make swift-crypto build on Windows (apple/swift-crypto#385)
* apple/swift-crypto@785709b Fix broken Windows tests (apple/swift-crypto#350) (apple/swift-crypto#386)
Steelskin added a commit to Steelskin/swift that referenced this pull request Jul 23, 2025
Steelskin added a commit to Steelskin/swift that referenced this pull request Jul 23, 2025
Steelskin added a commit to Steelskin/swift that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants