Building for MacOS should encourage setting an SDK version

[tresf]

HID4Java should be built using an older SDK, such as 10.9.

MacOS x86_64

• At time of writing this, the 10.9 SDK is the oldest SDK which Apple will allow notarization for.
• Normally the SDK can be extracted from a XCode 6.2 (apple.com).
• Alternately the SDK can be obtained as a stand-alone archive from a mirror such as: https://github.com/phracker/MacOSX-SDKs/releases/download/11.3/MacOSX10.9.sdk.tar.xz (much smaller file size)

MacOS aarch64

• At the time of writing this, the 11.0 SDK is the oldest SDK which is capable of building for the Apple Silicon/M1 aarch64 CPU.
• Normally the SDK can be extracted from a XCode 12.2 (apple.com)
• Alternately the SDK can be obtained as a stand-alone archive from a mirror such as: https://github.com/phracker/MacOSX-SDKs/releases/download/11.3/MacOSX11.0.sdk.tar.xz (much smaller file size)

Configuring the environment must be done in two steps:

• Specifying the full path to the SDK

  # e.g.
  export SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX10.9.sdk 

• Specifying the target OS

  # e.g.
  export MACOSX_DEPLOYMENT_TARGET=10.9

• To confirm, you can check the outputted file using otool -l

  # e.g.
  otool -l src/main/resources/darwin-x86-64/libhidapi.dylib
  # or (filter relevant portion for 10.9 or 11.0 binaries)
  otool -l src/main/resources/darwin-x86-64/libhidapi.dylib |grep -E 'LC_VERSION_MIN_MACOSX|LC_BUILD_VERSION' -A4

[gary-rowe]

Is this still an issue?

[tresf]

Is this still an issue?

It was merged here

• Improvements to MacOS builds #116

... However, I believe it's missing on the develop branch.

[tresf]

@gary-rowe which branch is this on? Or did you decide not to add it?

Here's an example of how this was done on another branch, but the code was never merged:

hid4java/build-hidapi.sh

Lines 330 to 334 in e40bdfe

	if [ -z "$SDKROOT" ] || [ -z "$MACOSX_DEPLOYMENT_TARGET" ];
	then
	echo -e "${yellow}WARNING: For production builds, please set \$SDKROOT and \$MACOSX_DEPLOYMENT_TARGET before building${plain}"
	echo -e "${yellow} See also https://github.com/gary-rowe/hid4java/issues/120${plain}"
	fi

[gary-rowe]

Hi @tresf . Looks like it was missed with all the other changes to the build-hidapi.sh script. I've just pushed some changes to develop which should address the minimum targets on macOS. I've tested the new .dylibs on a couple of macOS machines and they seem OK but if you'd like to check them over I'd appreciate it.

[tresf]

@gary-rowe thanks for the attention to detail however I think there may be some overkill items, such as restarting zsh with another architecture.

hid4java/build-hidapi.sh

Line 430 in 672f0a7

echo -e "${yellow} Re-run with 'arch -arm64 zsh ./build-hidapi.sh darwin-aarch64'${plain}"

... in my experience, this was just fine by passing the arch flag to the build system.

hid4java/build-hidapi.sh

Line 337 in e40bdfe

make CFLAGS="-arch x86_64"

I think this approach is cleaner and less confusing.