-
Notifications
You must be signed in to change notification settings - Fork 2
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 to use v1.1.0.1 SDK #20
Conversation
Remote build fails for arm64:
|
The changes on separate parts have been reverted due to building failure on arm64. Node.js and Zap installations need to be separated into a standalone part for better caching and clarity. I have created an issue for this purpose, which can be found #21. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes. At the same time with moving the zap build into a separate part, the new code should be refactored.
wget https://nodejs.org/dist/v12.22.12/node-v12.22.12-linux-x64.tar.xz | ||
tar xfvJ node-v12.22.12-linux-x64.tar.xz | ||
cp -rn node-v12.22.12-linux-x64/. /opt/node-v12.22.12-linux-x64/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was overlooked during my code review. This installs an x64 distribution of Nodejs on arm64.
Fix: #22
This PR might resolve #19
Testing
I have tested the matter-pi-gpio-commander snap built from this PR locally, with the chip-tool snap built from this PR, and they both work as expected using the same version of Matter SDK v1.1.0.1. The matter-pi-gpio-commander snap could be commissioned and controlled by chip-tool snap.
build:
install and setup matter-pi-gpio-commander:
install and setup chip-tool:
commison and control:
Upon successful execution, the LED on the Pi should be controlled by Chip Tool, turning it on or off.