Skip to content

gha/restructure llvmdev ci #1238

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

Merged
merged 11 commits into from
Jul 24, 2025
Merged

Conversation

swap357
Copy link
Contributor

@swap357 swap357 commented Jun 28, 2025

This PR introduces several key fixes to the llvmdev GHA build process.

  • adds user input options - 'all' for platforms and recipe. allows running all llvmdev workflows using one click/workflow dispatch. This also helps input only one workflow id on downstream llvmlite workflows.
  • builds llvmdev_for_wheel inside manylinux containers for linux-64 and linux-arm64 to ensure wheel compatibility.
  • pins the osx-64 compiler to version 14 to avoid conda selecting 17.x compilers.
  • fixes linux-arm64 builds by pinning py-lief to a version older than 0.16.

@esc
Copy link
Member

esc commented Jul 8, 2025

I think the osx-64 issue that needed a compiler pin has been resolved, can you remove those?

@esc
Copy link
Member

esc commented Jul 8, 2025

Can we use linux-aarch64 instead of linux-arm64 since the first is the name that conda uses?

@swap357
Copy link
Contributor Author

swap357 commented Jul 8, 2025

I think the osx-64 issue that needed a compiler pin has been resolved, can you remove those?

okay, will update.

Can we use linux-aarch64 instead of linux-arm64 since the first is the name that conda uses?

used -arm64 as its what gha uses but can change this. this will need changing downstream llvmlite workflow, will change there.

@swap357
Copy link
Contributor Author

swap357 commented Jul 9, 2025

keeping pin to use v14 compiler toolchain on both osx-64 and osx-arm64 for llvmdev and llvmdev-for-wheel recipes, since it otherwise gets 17.x that causes errors.

@seibert seibert merged commit d0d3d4b into numba:main Jul 24, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants