Skip to content

dev: Ensure latest setuptools used in CI#229

Open
adigitoleo wants to merge 6 commits into
mainfrom
fix-ci-setuptools
Open

dev: Ensure latest setuptools used in CI#229
adigitoleo wants to merge 6 commits into
mainfrom
fix-ci-setuptools

Conversation

@adigitoleo

Copy link
Copy Markdown
Member

It seems like the Python setup action in CI doesn't respect [build-system] in pyproject.toml ☹️ This ensures that the latest setuptools is being used in CI. I made the matching change to the virtual environment setup script.

Maybe this will resolve the failures to install llmvlite in the MacOS runner.

@adigitoleo

Copy link
Copy Markdown
Member Author

Well, the MacOS CI failure persists. The error message is confusing, as the logs show that setuptools v82 is being used, which is newer than the suggested v70.1, more investigation required.

  × Building wheel for llvmlite (pyproject.toml) did not run successfully.
140
  │ exit code: 1
141
  ╰─> [59 lines of output]
142
      /private/var/folders/yj/mflq9j197hz47cqbjv17r7k00000gn/T/pip-build-env-9ehaqa4p/overlay/lib/python3.13/site-packages/setuptools/_vendor/wheel/bdist_wheel.py:4: FutureWarning: The 'wheel' package is no longer the canonical location of the 'bdist_wheel' command, and will be removed in a future release. Please update to setuptools v70.1 or later which contains an integrated version of this command.

@adigitoleo

Copy link
Copy Markdown
Member Author

No luck on newer macos runner either. I don't think we want it to build llvmlite anyway should ideally just download a pre-built one? Might need to pin llmvlite version

@Patol75

Patol75 commented Mar 23, 2026

Copy link
Copy Markdown
Member

Sorry, I cannot manage to make it work either...

@adigitoleo

Copy link
Copy Markdown
Member Author

No worries I'll consider either pinning llvmlite (in pyproject.toml, kinda ugly as it then affects other platforms) or play around with adding brew insatll llvm before building the wheel on mac. Aside: my home office keyboard is dying so I will come back after a trip to JB 😆

@Patol75

Patol75 commented Mar 23, 2026

Copy link
Copy Markdown
Member

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