Skip to content
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

Build zim-tools and kiwix-tools on macos-arm64 #649

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Conversation

mgautierfr
Copy link
Member

@mgautierfr mgautierfr commented Nov 7, 2023

Fix #593
Fix #592

@mgautierfr
Copy link
Member Author

@rgaudin Can you try https://tmp.kiwix.org/ci/dev_preview/tools_on_macos_m1/kiwix-tools_macos-arm64-2023-11-07.tar.gz and https://tmp.kiwix.org/ci/dev_preview/tools_on_macos_m1/zim-tools_macos-arm64-2023-11-07.tar.gz ?

This PR removes the -fembed-bitcode option. I don't know the implication but it is also removed in #638 so I suppose we are ok.

@rgaudin
Copy link
Member

rgaudin commented Nov 8, 2023

Both work fine (tested kiwix-serve and zimcheck) 👍

Indeed, as explained, we want to remove bitcode from now

@mgautierfr mgautierfr changed the base branch from main to apple-xcf November 9, 2023 10:51
@mgautierfr
Copy link
Member Author

Rebased on #638 to use the changes made there.

@mgautierfr mgautierfr marked this pull request as ready for review November 9, 2023 10:53
@mgautierfr mgautierfr changed the title [WIP] Try to build zim-tools and kiwix-tools on macos-arm64 Build zim-tools and kiwix-tools on macos-arm64 Nov 9, 2023
Base automatically changed from apple-xcf to main November 10, 2023 15:01
@kelson42
Copy link
Contributor

@mgautierfr Is that ready to review by @rgaudin?

@mgautierfr
Copy link
Member Author

@mgautierfr Is that ready to review by @rgaudin?

It is, but the PR is really simple, most of the job has been made in #638.
I'm merging directly.

@mgautierfr mgautierfr merged commit 9b2b8c9 into main Nov 10, 2023
37 checks passed
@mgautierfr mgautierfr deleted the tools_on_macos_m1 branch November 10, 2023 17:50
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.

zim-tools should be published on macOS ARM Publish kiwix-tools on macOS
3 participants