NativeFileDialogSharp icon indicating copy to clipboard operation
NativeFileDialogSharp copied to clipboard

Add osx-arm64/native

Open kaylumah opened this issue 2 years ago • 6 comments

Add build/lib/Release/arm64/libnfd.dylib

Followed instructions from https://github.com/milleniumbug/NativeFileDialogSharp/issues/1#issuecomment-1474521250

kaylumah avatar Mar 18 '23 11:03 kaylumah

Would it be bloated to also add Linux arm64? :)

IceReaper avatar Apr 16 '23 10:04 IceReaper

Not much bloat involved to be honest, especially since users of the library can use appropriate options to not ship libraries for platforms they don't support;

The biggest issue that prevents me from merging this (and also adding support for linux-arm64) is that I have to recompile for every platform involved, as the osx-arm64 uses a newer version of the native library than the other platforms, and this will make for inconsistent behaviour between osx-x64 and osx-arm64.

Ideally this repo should

  • either have a CI/CD setup so it can adapt to potential future changes to the native library
  • or even delegate the responsibility of making shared libraries of the native code upstream

milleniumbug avatar Apr 16 '23 16:04 milleniumbug

Good point on the CI solution. that would be to prefer. Im basicaly trying to use this library here https://github.com/OpenRA/OpenRA/pull/20789 which is blocked mostly due to the missing natives currently.

IceReaper avatar Apr 17 '23 08:04 IceReaper

Could this be merged if we follow the same instruction on a osx-x64 or is it then still blocked for the diff between Mac and linux/windows?

kaylumah avatar Apr 20 '23 11:04 kaylumah

Been neglecting this one for a while, sorry about that.

Yeah, I'll need the osx-x64 version too as I lost my hackintosh virtual machine. I'll try to build Linux and Windows versions next weekend.

milleniumbug avatar Apr 15 '24 21:04 milleniumbug

No problem at all :) Unfortunately I no longer have a non-ARM Mac available, so I cannot provide the x64 MacOs binaries myself

kaylumah avatar Apr 16 '24 05:04 kaylumah