Skip to content

Reapply "Update build review" - #7

Merged
searchivarius merged 7 commits into
fast-pack:masterfrom
flaviomartins:update_build_1
Sep 11, 2026
Merged

Reapply "Update build review"#7
searchivarius merged 7 commits into
fast-pack:masterfrom
flaviomartins:update_build_1

Conversation

@flaviomartins

Copy link
Copy Markdown
Contributor

This reverts commit 6794a90.

(See: #5)

Now used as test-command for cibuildwheel
@searchivarius

Copy link
Copy Markdown
Collaborator

@flaviomartins thank you, it is ok to merge this one I assume?

@flaviomartins

Copy link
Copy Markdown
Contributor Author

Yes. CI is all green. Ready for merging.

@flaviomartins

Copy link
Copy Markdown
Contributor Author

To finish the job and upload/publish to pypi you will need to git tag v1.5.0 after merging

No compiled code actually calls the OpenMP runtime; -fopenmp merely forces the linker dependency, and auditwheel then bundles libgomp
@flaviomartins

flaviomartins commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

@searchivarius I managed to add in a few small commits before the merge. This is now ready to merge.

@searchivarius
searchivarius merged commit c423152 into fast-pack:master Sep 11, 2026
7 checks passed
@flaviomartins

Copy link
Copy Markdown
Contributor Author

To finish the job and upload/publish to pypi you will need to git tag v1.5.0 after merging

@searchivarius you might need to git push --tags as well :-)

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