Skip to content

Conversation

@compnerd
Copy link
Member

@compnerd compnerd commented Nov 5, 2020

Reverts #34557

This seems to have caused a regression on Windows: https://ci-external.swift.org/job/oss-swift-windows-x86_64-vs2019/3362/consoleText

@compnerd
Copy link
Member Author

compnerd commented Nov 5, 2020

@swift-ci please smoke test

@compnerd
Copy link
Member Author

compnerd commented Nov 5, 2020

CC: @zoecarver @hlopko

Copy link
Contributor

@zoecarver zoecarver left a comment

Choose a reason for hiding this comment

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

@compnerd sorry about that. I will try to re-land later today. Thanks for reverting.

When I ran swift-ci smoke test and merge it didn't trigger the Windows bots. Is that intentional or a bug?

@compnerd compnerd merged commit 91b969a into main Nov 5, 2020
@compnerd compnerd deleted the revert-34557-cxx/fix/is-dependent-type-over-aligned branch November 5, 2020 18:18
@compnerd
Copy link
Member Author

compnerd commented Nov 5, 2020

@zoecarver that's a bug, @shahmishal might be able to identify why exactly it failed to trigger in the particular case.

@shahmishal
Copy link
Member

This is known issue because windows is part of different infrastructure (Swift Community-Hosted CI vs Swift CI).

@zoecarver
Copy link
Contributor

Good to know. In the future, I'll just do a smoke test and merge manually then.

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.

4 participants