search

utilityai / llama-cpp-rs

166 stars 49 forks source link

build.rs: also search for library files in lib64/ #561

Closed AsbjornOlling closed 3 weeks ago

AsbjornOlling commented 3 weeks ago

This fixes compilation on my machine and in Nix.

It seems like libllama.a ends up in ${out_dir}/lib64/, and not in ${out_dir}/lib/, like the build script assumes.

Why? I'm not quite sure. But it hapens on the current nixos-unstable, using cargo, cmake and clang from nixpkgs.

It also affects building in the nix build sandbox (using e.g. nixpkgs.rustPlatofrm.buildRustPackage), so after this patch it is again possible to build rust applications that depend on this library using the nix build system.

MarcusDunn commented 3 weeks ago

Bonus points to add an error message to the assert_eq. Otherwise looks great. Thanks for the PR. I will merge once tests look mostly green.