Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ZAP version for Matter SDK master branch #41

Closed
MonicaisHer opened this issue Jan 29, 2024 · 2 comments · Fixed by #47
Closed

Upgrade ZAP version for Matter SDK master branch #41

MonicaisHer opened this issue Jan 29, 2024 · 2 comments · Fixed by #47
Labels
bug Something isn't working

Comments

@MonicaisHer
Copy link
Contributor

As the Matter SDK branch has been switched to master, the ZAP version is now outdated, leading to build failures in the edge channel. The ZAP version needs to be updated according to the MIN_ZAP_VERSION.

@farshidtz
Copy link
Member

farshidtz commented Jan 30, 2024

Build error from 4 days ago:

:: [58/674] ACTION //third_party/connectedhomeip/src/platform/Linux/dbus/wpa:wpa_gen(//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc)
:: [59/674] ACTION //third_party/connectedhomeip/src/controller/data_model:data_model_zapgen_zap_pregen(//third_party/connectedhomeip/build/toolchain/linux:linux_arm64_gcc)
:: FAILED: gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated/access.h gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated/gen_config.h gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated/endpoint_config.h gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated/CHIPClusters.h gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated/IMClusterCommandHandler.cpp
:: python ../examples/chip-tool/third_party/connectedhomeip/third_party/pigweed/repo/pw_build/py/pw_build/python_runner.py --gn-root ../examples/chip-tool/ --current-path ../examples/chip-tool/third_party/connectedhomeip/src/controller/data_model --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/src/controller/data_model/data_model_zapgen_zap_pregen_metadata_path_list.txt -- ../examples/chip-tool/third_party/connectedhomeip/scripts/tools/zap/generate.py --no-prettify-output --templates /build/chip-tool/parts/connectedhomeip/src/examples/chip-tool/third_party/connectedhomeip/src/app/zap-templates/app-templates.json --output-dir /build/chip-tool/parts/connectedhomeip/src/build-examples/gen/third_party/connectedhomeip/src/controller/data_model/zap_pregen/zap-generated --lock-file /build/chip-tool/parts/connectedhomeip/src/build-examples/zap_gen.lock --parallel /build/chip-tool/parts/connectedhomeip/src/examples/chip-tool/third_party/connectedhomeip/src/controller/data_model/controller-clusters.zap
:: Searching for zcl file from /build/chip-tool/parts/connectedhomeip/src/examples/chip-tool/third_party/connectedhomeip/src/controller/data_model/controller-clusters.zap
:: Checking ZAP from ['/build/chip-tool/parts/zap/build/zap-cli']:
::   !!! Version validation failed: required at least 2023.12.6, got 2023.11.10 instead

@farshidtz farshidtz added the bug Something isn't working label Jan 30, 2024
@farshidtz
Copy link
Member

Since project-chip/connectedhomeip#26636, the SDK automatically downloads a compatible ZAP version for arm64.

Moreover, it is recommended to programmatically download the compatible ZAP version by looking into one of zap.version or zap.json, rather than hard cording a version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants