-
Notifications
You must be signed in to change notification settings - Fork 10.5k
The main branch windows CI failing (missing mimalloc checkout) #79115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The main branch windows CI failing (missing mimalloc checkout) #79115
Comments
@shahmishal can you add the mimalloc repo https://github.com/microsoft/mimalloc as a dependency of the windows toolchain builds on the ci-external.swift.org CIs? |
This is now fixed, I just updated the CI config. |
Cannot confirm yet, because there are no recent runs |
Unfortunately, it still fails: https://ci-external.swift.org/job/swift-main-windows-toolchain/1029/
|
@shahmishal Should I land the revert linked above? |
mimalloc seems to be available now on the mainline build bots:
PRs #79151 and swiftlang/swift-installer-scripts#381 are about to re-land. |
Description
https://ci-external.swift.org/job/swift-main-windows-toolchain/
https://ci-external.swift.org/job/swift-main-windows-toolchain-arm64/
https://ci-external.swift.org/job/swift-main-windows-toolchain/1026/consoleText
I think that the root cause is the fact that the mimalloc repo isn't added to the dependencies (it's not checked out during the build).
Revert #79112
Probably this needs to be reverted as well: swiftlang/swift-installer-scripts#371
CC @shahmishal @compnerd @weliveindetail
Reproduction
CI
Expected behavior
No failure
Environment
CI
Additional information
No response
The text was updated successfully, but these errors were encountered: