Skip to content

Conversation

@etcwilde
Copy link
Member

Only the Apple-platforms support dsymutil and building/linking with the swiftmodule, therefore, this test only applies to running on macOS where we have dsymutil. I've also cleaned up the test a bit to make it a bit clearer what the goals were.
We're trying to verify that the contents of the filelist passed into the link job contains only the expected object file and does not include the swiftmodule file.

Fixes: rdar://144378596

Only the Apple-platforms support dsymutil and building/linking with the swiftmodule, therefore, this test only applies to running on macOS where we have dsymutil. I've also cleaned up the test a bit to make it a bit clearer what the goals were. We're trying to verify that the contents of the filelist passed into the link job contains only the expected object file and does not include the swiftmodule file. Fixes: rdar://144378596
@etcwilde etcwilde requested a review from artemcm February 12, 2025 21:03
@etcwilde
Copy link
Member Author

@swift-ci please test

@etcwilde
Copy link
Member Author

Windows Test Failure:

C:\Users\swift-ci\jenkins\workspace\swift-driver-PR-windows\swift-testing\Sources\Testing\Support\JSON.swift:44:28: error: type 'Any' does not conform to the 'Sendable' protocol 42 | 43 | // Set user info keys that clients want to use during encoding. 44 | encoder.userInfo.merge(userInfo, uniquingKeysWith: { _, rhs in rhs}) | `- error: type 'Any' does not conform to the 'Sendable' protocol 45 | 46 | let data = try encoder.encode(value) 
@etcwilde
Copy link
Member Author

@swift-ci please test Windows

1 similar comment
@etcwilde
Copy link
Member Author

@swift-ci please test Windows

@etcwilde etcwilde merged commit ad8a432 into swiftlang:main Feb 13, 2025
3 checks passed
@etcwilde etcwilde deleted the ewilde/macos-debuginfo-filelists branch February 13, 2025 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants