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

Build failure: python3Packages.faiss (on Darwin) #340842

Closed
sarahec opened this issue Sep 9, 2024 · 6 comments
Closed

Build failure: python3Packages.faiss (on Darwin) #340842

sarahec opened this issue Sep 9, 2024 · 6 comments
Labels

Comments

@sarahec
Copy link
Contributor

sarahec commented Sep 9, 2024

Steps To Reproduce

  1. On Darwin...
  2. build python312Packages.faiss
  3. Observe:
make: *** [Makefile:208: swigfaiss] Error 2
error: builder for '/nix/store/mbr9lq6ppqzws7q2qv6s70fhrhrzjl3z-faiss-1.8.0.drv' failed with exit code 2;
       last 10 log lines:
       >         return SWIG_NewPointerObj(data, SWIGTYPE_p_unsigned_long, 0);
       >                                         ^
       > /tmp/nix-build-faiss-1.8.0.drv-0/source/build/faiss/python/CMakeFiles/swigfaiss.dir/swigfaissPYTHON_wrap.cxx:5095:41: error: use of undeclared identifier 'SWIGTYPE_p_long'
       >         return SWIG_NewPointerObj(data, SWIGTYPE_p_long, 0);
       >                                         ^
       > 2 errors generated.
       > make[3]: *** [faiss/python/CMakeFiles/swigfaiss.dir/build.make:76: faiss/python/CMakeFiles/swigfaiss.dir/CMakeFiles/swigfaiss.dir/swigfaissPYTHON_wrap.cxx.o] Error 1
       > make[2]: *** [CMakeFiles/Makefile2:269: faiss/python/CMakeFiles/swigfaiss.dir/all] Error 2
       > make[1]: *** [CMakeFiles/Makefile2:276: faiss/python/CMakeFiles/swigfaiss.dir/rule] Error 2
       > make: *** [Makefile:208: swigfaiss] Error 2
       For full logs, run 'nix log /nix/store/mbr9lq6ppqzws7q2qv6s70fhrhrzjl3z-faiss-1.8.0.drv'.
error: 1 dependencies of derivation '/nix/store/wnjfqxjfakg3m3ymdf9bf661zm3rzxq2-python3.12-faiss-1.8.0.drv' failed to build

This is a known issue when swig generates Darwin code: swig/swig#2864
Faiss accepted a patch for the next release: facebookresearch/faiss@bd22c93

Notify maintainers

@SomeoneSerge
cc: @natsukium

Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.6.0, macOS 14.6.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.18.5`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Add a 👍 reaction to issues you find important.

@dotlambda dotlambda added 6.topic: python 6.topic: darwin Running or building packages on Darwin labels Sep 9, 2024
@sarahec
Copy link
Contributor Author

sarahec commented Sep 9, 2024

Unfortunately, cherry-picking the patch didn't work. I'll close this for now as I don't see a way to patch around it; I'll see if I can lobby for an interim faiss release.

@sarahec sarahec closed this as completed Sep 9, 2024
@SomeoneSerge
Copy link
Contributor

So the issue is still present? Then we should keep this open?

@sarahec
Copy link
Contributor Author

sarahec commented Sep 10, 2024

Yes, it's still present so let's keep this open.

I've asked the faiss team if they could cut a release to include Darwin. We'll see what they say.

Otherwise, I'll keep looking into potential patches.

@sarahec sarahec reopened this Sep 10, 2024
@sarahec
Copy link
Contributor Author

sarahec commented Sep 11, 2024

See facebookresearch/faiss#3846

@sarahec
Copy link
Contributor Author

sarahec commented Sep 17, 2024

@SomeoneSerge they expect to produce a new release next week.

@sarahec
Copy link
Contributor Author

sarahec commented Oct 11, 2024

Closed by #341418

@sarahec sarahec closed this as completed Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants