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

ci: use uv in cibuildwheel #81

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

ci: use uv in cibuildwheel #81

wants to merge 2 commits into from

Conversation

henryiii
Copy link
Collaborator

No description provided.

@henryiii
Copy link
Collaborator Author

Any idea why:

src/Makefile.am:2: error: Libtool library used but 'LIBTOOL' is undefined
src/Makefile.am:2:   The usual way to define 'LIBTOOL' is to add 'LT_INIT'
src/Makefile.am:2:   to 'configure.ac' and run 'aclocal' and 'autoconf' again.
src/Makefile.am:2:   If 'LT_INIT' is in 'configure.ac', make sure
src/Makefile.am:2:   its definition is in aclocal's search path.

Is happening? The only difference should be python -m build --wheel is used instead of pip wheel ., and uv set up the Python environments instead of Pip (and a copy of pip is not installed).

@kratsg
Copy link
Owner

kratsg commented Jun 11, 2024

Is happening? The only difference should be python -m build --wheel is used instead of pip wheel ., and uv set up the Python environments instead of Pip (and a copy of pip is not installed).

Nope. The makefile in question comes from https://github.com/file/file. Maybe one needs to add sudo apt-get install libtool or similar?

EDIT: the reason I say this is the mac wheels were fine, and that explicitly installs libtool. So perhaps the virtualenv is not seeing the system-wide libtool that is on the Linux runners, somehow? Or that pip shipped with libtool (I know it's part of pip) while it's not part of the wheel by default...

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