Skip to content

Don't attempt to generate DSYMs on non-darwin platforms#607

Merged
owenv merged 1 commit into
mainfrom
owenv/linux-dsym-fix
Jun 24, 2025
Merged

Don't attempt to generate DSYMs on non-darwin platforms#607
owenv merged 1 commit into
mainfrom
owenv/linux-dsym-fix

Conversation

@owenv

@owenv owenv commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator

Addresses swiftlang/swift-package-manager#8862

Don't try to generate a dSYM on non-Darwin platforms

@owenv owenv force-pushed the owenv/linux-dsym-fix branch 2 times, most recently from 26aec2d to f215f41 Compare June 23, 2025 21:47
@owenv

owenv commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test

@owenv owenv force-pushed the owenv/linux-dsym-fix branch from f215f41 to 71507d0 Compare June 23, 2025 23:16
@owenv

owenv commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test

@owenv owenv marked this pull request as ready for review June 23, 2025 23:16
@owenv

owenv commented Jun 24, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test Windows

@owenv

owenv commented Jun 24, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test macOS

@owenv owenv merged commit eb6551d into main Jun 24, 2025
24 checks passed
@owenv owenv deleted the owenv/linux-dsym-fix branch June 24, 2025 21:01
cmcgee1024 added a commit to swiftlang/swift-package-manager that referenced this pull request Jun 27, 2025
#8879)

Until[swift-build#607](swiftlang/swift-build#607) was
merged, there was an error due to a missing dsymutil tool on non-macOS
system, such as Linux. Remove the skip from the tests now that this is working.

Fixes #8862
marcprux pushed a commit to swift-everywhere/swift-package-manager that referenced this pull request Jul 7, 2025
swiftlang#8879)

Until[swift-build#607](swiftlang/swift-build#607) was
merged, there was an error due to a missing dsymutil tool on non-macOS
system, such as Linux. Remove the skip from the tests now that this is working.

Fixes swiftlang#8862
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.

2 participants