Skip to content

Conversation

@marek-elisity
Copy link

No description provided.

@marek-elisity marek-elisity mentioned this pull request Jun 17, 2025
@marek-elisity
Copy link
Author

@escherba, any chances to get an Alpine support here? I'm trying to update the CI to build it, but it fails miserably :/

@escherba
Copy link
Owner

escherba commented Aug 2, 2025

Hi Marek, I have two comments: (1) I already addressed the issue with 128-bit functions returning long ints in master branch (#96), so rebasing should do the trick. (2) it appears you tried to modify "build.yml" workflow to actually build wheels for distribution. That is not the purpose of that workflow, that workflow exists simply to verify newly created PRs. There is another workflow file "publish.yml" that actually does the job of building and distributing wheels. Sorry if i caused confusion with naming, perhaps i should rename "build.yml" to "check-build.yml" or something similar.

That said I do not actually know what building wheels for Alpine requires. Maybe you could link some pointers for me to work with.

@escherba
Copy link
Owner

escherba commented Aug 2, 2025

The following command now runs successfully for me: docker run --rm python:3.11-alpine sh -c "apk add --no-cache build-base && pip install cityhash". Closing this issue.

@escherba escherba closed this Aug 2, 2025
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.

2 participants