-
Notifications
You must be signed in to change notification settings - Fork 54
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
nanobind editable install fails due to mishandled .DS_Store
#961
Comments
One curious thing is why is it being installed as editable? Maybe this is part of Anyway for the current issue, an immediate solution would be to just strip the |
Preliminary issue that I found: nanobind has a file The issue is probably when it is trying to resolve: scikit-build-core/src/scikit_build_core/build/wheel.py Lines 90 to 113 in 9170ab8
Could you also run the installation with verbose logging? See https://scikit-build-core.readthedocs.io/en/latest/configuration.html#verbosity |
That's just the default of Running with
|
Ok so Some more digging Ultimately though I think |
But you're spot on I think. Putting
So it seems .DS_Store is indeed the culprit. How does that slip through though, if it's in gitignore? |
.DS_Store
Ok, things are more complicated because the file is installed because of the |
Thanks. Could you explain why this fails in editable mode only, and not in a proper install? |
If you do a |
Nope, doesn't fail with |
I thought I'd raise this here, please excuse if this is misplaced.
My system:
brew install uv
.Repro:
uv venv --seed -p 3.13
.source .venv/bin/activate
.uv sync
.Output:
Looking more closely at the top and bottom of the traceback (the
wheel_filename = backend.build_editable("/Users/nicholasjunge/Library/Caches/uv/builds-v0/.tmpgtK0HP", {}, None)
call), scikit-build-core is dealt a tempdir by uv containing a dot to start in its name, which it doesn't expect (or at least, pathlib cannot handle).The text was updated successfully, but these errors were encountered: