search

canonical / matter-pi-gpio-commander

Matter Raspberry Pi GPIO Commander - Turn your Pi into a Matter lighting device!
Apache License 2.0
90 stars 3 forks source link

Compatibility problems on `ARM64` builds #54

Closed locnnil closed 3 months ago

locnnil commented 3 months ago

The build of main branch is failing on arm64.

As showed in this action run:

Something made it incompatible with the upstream chip project, similar to what happens on #43.

The errors: 

:: [69/639] ACTION //third_party/connectedhomeip/examples/lighting-app/lighting-common:lighting-common_codegen_generate(//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc)
:: [70/639] ACTION //third_party/connectedhomeip/examples/lighting-app/lighting-common:lighting-common_zapgen_generate(//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc)
:: FAILED: gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/zapgen/zap-generated/access.h gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/zapgen/zap-generated/gen_config.h gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/zapgen/zap-generated/endpoint_config.h gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/zapgen/zap-generated/IMClusterCommandHandler.cpp
:: python ../../third_party/connectedhomeip/third_party/pigweed/repo/pw_build/py/pw_build/python_runner.py --gn-root ../../ --current-path ../../third_party/connectedhomeip/examples/lighting-app/lighting-common --default-toolchain=//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc --current-toolchain=//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc --capture-output --python-dep-list-files gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/lighting-common_zapgen_generate_metadata_path_list.txt -- ../../third_party/connectedhomeip/scripts/tools/zap/generate.py --no-prettify-output --templates /root/parts/connectedhomeip/src/examples/lighting-app/linux/third_party/connectedhomeip/src/app/zap-templates/app-templates.json --output-dir /root/parts/connectedhomeip/src/examples/lighting-app/linux/out/build/gen/third_party/connectedhomeip/examples/lighting-app/lighting-common/zapgen/zap-generated --lock-file /root/parts/connectedhomeip/src/examples/lighting-app/linux/out/build/zap_gen.lock --parallel /root/parts/connectedhomeip/src/examples/lighting-app/linux/third_party/connectedhomeip/examples/lighting-app/lighting-common/lighting-app.zap
:: Searching for zcl file from /root/parts/connectedhomeip/src/examples/lighting-app/linux/third_party/connectedhomeip/examples/lighting-app/lighting-common/lighting-app.zap
:: Checking ZAP from ['/root/parts/zap/build/zap-cli']:
::   !!! Version validation failed: required at least 2024.3.14, got 2024.1.5 instead
:: [71/639] stamp obj/third_party/connectedhomeip/examples/lighting-app/lighting-common/lighting-common_codegen_generate.stamp
:: [72/639] ar openthread.a
farshidtz commented 3 months ago

It just needs a version bump for zap. The error message shows which version is needed.

A better approach would be to remove the zap part and all associated code in snapcraft.yaml, because the SDK now pulls the compatible zap artifact for arm64. This was recently removed from chip tool snap: https://github.com/canonical/chip-tool-snap/pull/47