Skip to content

[build] invoke swift-serialize-diagnostics for host architecture - #33402

Merged
edymtt merged 2 commits into
swiftlang:masterfrom
edymtt:call-swift-serialize-diagnostics-from-host
Aug 11, 2020
Merged

[build] invoke swift-serialize-diagnostics for host architecture#33402
edymtt merged 2 commits into
swiftlang:masterfrom
edymtt:call-swift-serialize-diagnostics-from-host

Conversation

@edymtt

@edymtt edymtt commented Aug 11, 2020

Copy link
Copy Markdown
Contributor

As a result of #33346 we are relying on the tool for the current architecture -- this will
work for the host architecture but will fail when crosscompiling.

Addresses rdar://66800239

As a result of swiftlang#33346 we are relying on the tool for the current target -- this will
work for the host architecture but will fail when crosscompiling.

Addresses rdar://66800239
@edymtt

edymtt commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

@swift-ci please smoke test

@shahmishal shahmishal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@shahmishal

Copy link
Copy Markdown
Member
00:15:04.027 /bin/sh: /Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/branch-master/buildbot_incremental/swift-macosx-x86_64/./bin/swift-serialize-diagnostics: No such file or directory

@edymtt

edymtt commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

@swift-ci please smoke test

@edymtt

edymtt commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

@swift-ci please test Windows platform

@edymtt

edymtt commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

@swift-ci please clean test Windows platform

@xedin xedin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

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.

3 participants