Skip to content

Conversation

@daveinglis
Copy link
Contributor

needs to be investigated

rdar://160781783

@daveinglis
Copy link
Contributor Author

@swift-ci test

@daveinglis
Copy link
Contributor Author

@swift-ci test linux

@daveinglis daveinglis changed the title mark a failing test on linux as withKnowIssue (moduleDirectDeps2) mark a failing test on linux+swiftbuild as withKnownIssue (moduleDirectDeps2) Sep 17, 2025
…ps2)

This was previously marked as a known Issue before but got changed in
another PR resulting in a PR merge race causing breakage.
@daveinglis
Copy link
Contributor Author

@swift-ci test

@daveinglis
Copy link
Contributor Author

@swift-ci test windows

@daveinglis daveinglis enabled auto-merge (squash) September 18, 2025 13:57
}
} when: {
ProcessInfo.hostOperatingSystem == .windows && buildSystem == .swiftbuild
buildSystem == .swiftbuild
Copy link
Contributor

@bkhouri bkhouri Sep 18, 2025

Choose a reason for hiding this comment

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

do we understand what started to cause this failure as the change that added support for module aliasing in SwiftBuild has passed on all platforms, except Windows due to known issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, we need to fix SwiftBuild linker task to specify linked libraries correctly. This will be my next update soon

@bnbarham
Copy link
Contributor

Merging to unblock PR testing

@bnbarham bnbarham disabled auto-merge September 18, 2025 16:18
@bnbarham bnbarham merged commit d608e96 into swiftlang:main Sep 18, 2025
2 of 6 checks passed
@daveinglis daveinglis deleted the linux_test_fix branch October 2, 2025 18:14
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.

5 participants