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

[spec] tv app cluster definitions not up to spec #19174

Closed
andy31415 opened this issue Jun 3, 2022 · 1 comment
Closed

[spec] tv app cluster definitions not up to spec #19174

andy31415 opened this issue Jun 3, 2022 · 1 comment
Assignees

Comments

@andy31415
Copy link
Contributor

Problem

./scripts/idl_lint.py --log-level warn ./examples/tv-app/tv-common/tv-app.matter
2022-06-03 10:15:34 ERROR   ERROR: Required attributes: EP0:GroupKeyManagement does not expose MaxGroupsPerFabric(2) attribute at ./examples/tv-app/tv-common/tv-app.matter:2289:3
2022-06-03 10:15:34 ERROR   ERROR: Required attributes: EP0:GroupKeyManagement does not expose MaxGroupKeysPerFabric(3) attribute at ./examples/tv-app/tv-common/tv-app.matter:2289:3
2022-06-03 10:15:34 ERROR   ERROR: Required attributes: EP5:ApplicationBasic does not expose AllowedVendorList(7) attribute at ./examples/tv-app/tv-common/tv-app.matter:2541:3
2022-06-03 10:15:34 ERROR   ERROR: Required commands: Cluster EthernetNetworkDiagnostics does not define mandatory command ResetCounts(0) at ./examples/tv-app/tv-common/tv-app.matter:526:1
2022-06-03 10:15:34 ERROR   ERROR: Required commands: Cluster GroupKeyManagement does not define mandatory command KeySetWrite(0) at ./examples/tv-app/tv-common/tv-app.matter:775:1
2022-06-03 10:15:34 ERROR   ERROR: Required commands: Cluster GroupKeyManagement does not define mandatory command KeySetRead(1) at ./examples/tv-app/tv-common/tv-app.matter:775:1
2022-06-03 10:15:34 ERROR   ERROR: Required commands: Cluster GroupKeyManagement does not define mandatory command KeySetRemove(3) at ./examples/tv-app/tv-common/tv-app.matter:775:1
2022-06-03 10:15:34 ERROR   ERROR: Required commands: Cluster GroupKeyManagement does not define mandatory command KeySetReadAllIndices(4) at ./examples/tv-app/tv-common/tv-app.matter:775:1
2022-06-03 10:15:34 ERROR   Found 8 lint errors
@lazarkov
Copy link
Contributor

lazarkov commented Jun 7, 2022

@andy31415 FYI running python from matter environment worked out of the box:.environment/pigweed-venv/bin/python3 scripts/idl_lint.py --log-level warn ./examples/tv-app/tv-common/tv-app.matter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants