slinc icon indicating copy to clipboard operation
slinc copied to clipboard

Aarch64 support

Open markehammons opened this issue 1 year ago • 5 comments

As a modern mac user, I would like slinc to support AArch64.

This story is complete when sprintf works on aarch64 machines.

markehammons avatar Mar 27 '23 20:03 markehammons

As a note, I just bumped into this now, whilst taking my experiment home. It looks like this is expected behaviour for the time being.

Exception in thread "main" java.lang.Error: No library file found for AArch64 Darwin with path like /Users/simon/Code/PackageCompiler.jl/examples/MyLibCompiled/lib/libjulia.dylib
        at fr.hammons.slinc.modules.LinkageTools$.$anonfun$7(LinkageTools.scala:108)

Happy to guinea pig something if that's helpful? My "real" use case is on Windows however....

Quafadas avatar Apr 11 '23 19:04 Quafadas

I will put this in the product backlog, but it's a bit of a difficult issue for me as I have no M1 to test on. I'll look into having a runner for the CI on AWS, but that's real low priority for me as there's more pressing issues to work on.

markehammons avatar Apr 12 '23 05:04 markehammons

One note, even for mac I'm using .so instead of .dylib. I'm guessing this should change based on you use-case.

markehammons avatar Apr 12 '23 08:04 markehammons

If you do not provide an absolute file name (file ending with .so or .dll), Slinc will use the base file name provided along with the appropriate library suffix for the OS and a tag based on the architecture. https://slinc.hammons.fr/docs/docs/reference/library-modules.html#

Can we regard file ending with .dylib as absolute file name too?

This is just an idea, but I think it would be nice to be able to specify a file name without conversion. I mean something like the following example.

@NeedsFile("build/name") // => converted into /path/to/name_{suffix}.so, path/to/name_{suffix}.dll
@NeedsFile("`build/name`") //=> path/to/name

i10416 avatar Apr 14 '23 07:04 i10416

@i10416 lets discuss that further here and leave this issue just for AArch64 support specifically.

markehammons avatar Apr 14 '23 12:04 markehammons