search

terminusdb-labs / swipl-rs

25 stars 6 forks source link

Small fix to support aarch64. #35

Closed blueforesticarus closed 1 year ago

blueforesticarus commented 1 year ago

Tests pass on both x86 and arm after the change. To get tests and cargo-swipl to build nixos I also had to specify the link path like so

RUSTFLAGS="-L/nix/store/8qslsmfg3b6y35r2dmsycf65vpk8msma-swi-prolog-8.3.29/lib/"
RUSTDOCFLAGS="-L/nix/store/8qslsmfg3b6y35r2dmsycf65vpk8msma-swi-prolog-8.3.29/lib/"

but I expect on a regular system they will just work.

I was able to patch this into terminusdb and everything appears to be working.

matko commented 1 year ago

Thanks a bunch! Interesting to see that chars are apparently not i8 on aarch, or at least not generated as such by bindgen.

@rrooij it'd be nice if we could have aarch in our CI, cause there's a big chance I'm inevitably going to screw this up again, given that i do not have any aarch hardware. Is that possible?