-
Notifications
You must be signed in to change notification settings - Fork 25
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
Try CI on macOS 13 explicitly. #1760
Conversation
b04ab07
to
e6144e3
Compare
cfd4a5e
to
915dad8
Compare
45e44d8
to
e0a1be7
Compare
ab28569
to
36d945e
Compare
f5247c0
to
71314b5
Compare
This reverts commit dd436f9.
This run for iOS was made with the Nimble v11 (the commit unfortunately was lost because of force push), and failed tests are the same as in tvOS run with the Nimble v9. This run shows that it doesn't depend on Xcode version (fails with similar set of tests on Xcode 14.2), so it's macOS 13 thing. @lawrence-forooghian Couldn't you please check these set of tests locally?
|
I'll try running on Xcode 14.2 on macOS 13. But I'm a bit confused — if you've seen the same test failures on tvOS 16.4 (which is what I understand from #1760 (comment)) then what makes you think it's a "macOS 13 thing"? |
I ran the macOS tests on commit 01b0eb7, on my machine with macOS 13.4.1, with Xcode 14.2, and did not get any test failures. |
It's because that run under tvOS has an invalid setup - tvOS 16.4 on Xcode 14.2. Shouldn't be the case. So yeah, it's a Xcode 14.3 thing (including macOS 13 itself run) |
Another set of experiments finally showed that this depends only on whether it's on CI macOS 13. The same tests on macOS, iOS and tvOS failed under macOS 13 and Xcode 14.2 (they are fine on Xcode 14.2 under macOS 12). Here are the same failures under Xcode 14.3. Also here the same failures on macOS 13 and Xcode 14.3 for tvOS, but with old Nimble (tvOS compiles in Xcode 14.3 without need to update Nimble for some reason). All those tests succeed locally on macOS 13 and Xcode 14.3 for all platforms. @lawrence-forooghian |
This work is done in #1801 |
Not for merging