Skip to content

Conversation

@kelson42
Copy link
Contributor

@kelson42 kelson42 commented Apr 26, 2025

Move the whole CI to macOS 15, which implies updated compilation stack and full (botch arch+system) cross-compilation for x86_64 arch.

@kelson42 kelson42 added this to the 9.4.0 milestone Apr 26, 2025
@codecov
Copy link

codecov bot commented Apr 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.13%. Comparing base (c883006) to head (6f8d648).
⚠️ Report is 23 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #980   +/-   ##
=======================================
  Coverage   58.13%   58.13%           
=======================================
  Files         101      101           
  Lines        5384     5384           
  Branches     2197     2197           
=======================================
  Hits         3130     3130           
  Misses        795      795           
  Partials     1459     1459           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kelson42 kelson42 force-pushed the add-macos15-ci branch 6 times, most recently from adc7bfb to 1923a1e Compare May 2, 2025 05:30
@kelson42 kelson42 marked this pull request as draft May 2, 2025 05:32
@kelson42 kelson42 force-pushed the add-macos15-ci branch 3 times, most recently from 7be28e5 to 4f5cd3c Compare June 5, 2025 09:08
@kelson42
Copy link
Contributor Author

kelson42 commented Jun 5, 2025

@rgaudin Any idea? Do we have a path problem? Or should we compile with Xcode16? https://github.com/openzim/libzim/actions/runs/15463093103/job/43528832086?pr=980#step:9:58

@rgaudin
Copy link
Member

rgaudin commented Jun 5, 2025

Message is pretty clear:

meson.build:1:0: ERROR: Unknown compiler(s): [['/Applications/Xcode_15.0.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang']]
The following exception(s) were encountered:
Running `/Applications/Xcode_15.0.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang --version` gave "[Errno 2] No such file or directory: '/Applications/Xcode_15.0.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang'"

As you can see above, it's configured for Xcode_15.0.1 but macos-15 only includes Xcode 16.x.

This is I believe due to the previous step (Install dependencies). It downloads and uses https://tmp.kiwix.org/ci/deps_macos-aarch64-dyn_libzim.tar.gz which meson_cross_file.txt references that 15.0.1

@kelson42
Copy link
Contributor Author

kelson42 commented Jun 5, 2025

@rgaudin Does that means we need to move to Xcode 16 at kiwix/kiwix-build first?

@rgaudin
Copy link
Member

rgaudin commented Jun 5, 2025

I guess. I don't think this external dependencies behavior can work with different Xcode versions.

@kelson42
Copy link
Contributor Author

kelson42 commented Jun 5, 2025

Blocked by kiwix/kiwix-build#830

@kelson42 kelson42 force-pushed the add-macos15-ci branch 2 times, most recently from 0eb8b60 to 75bf976 Compare October 8, 2025 10:03
@kelson42
Copy link
Contributor Author

kelson42 commented Oct 8, 2025

@rgaudin Please let me know if something could be improved (or maybe just patch directly the PR) as this will be use to upgrade macOS CI for other projects.

@kelson42 kelson42 marked this pull request as ready for review October 8, 2025 10:17
Copy link
Member

@rgaudin rgaudin left a comment

Choose a reason for hiding this comment

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

LGTM ; seems to be working fine. I checked the deps archives and the logs. 👍

@rgaudin rgaudin changed the title Add macOS 15 to CI Use macOS 15 for CI Oct 8, 2025
@kelson42 kelson42 merged commit 7e5784c into main Oct 8, 2025
28 checks passed
@kelson42 kelson42 deleted the add-macos15-ci branch October 8, 2025 12:01
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