-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Chip tool interactive #16135
Chip tool interactive #16135
Conversation
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.
Fast tracking given this has had > 3 days for review.
/rebase |
1e0a796
to
729e520
Compare
PR #16135: Size comparison from 125e73c to 729e520 Increases above 0.2%:
Increases (1 build for linux)
Full report (25 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
729e520
to
d10d10d
Compare
b207340
to
6e98f2c
Compare
PR #16135: Size comparison from 08906fb to 6e98f2c Increases above 0.2%:
Increases (1 build for linux)
Full report (25 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
6e98f2c
to
e2f2d65
Compare
PR #16135: Size comparison from 08906fb to e2f2d65 Increases above 0.2%:
Increases (1 build for linux)
Full report (25 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
8c553b9
to
c3c1b68
Compare
PR #16135: Size comparison from 08906fb to c3c1b68 Increases above 0.2%:
Increases (1 build for linux)
Full report (23 builds for cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, telink)
|
c3c1b68
to
b8e8465
Compare
PR #16135: Size comparison from b80df3d to b8e8465 Increases above 0.2%:
Increases (1 build for linux)
Full report (26 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
* [chip-tool] Add chip-tool interactive mode * Update generated content * Allow interactive mode to be disabled for some build variants
…t as a requirement
* [chip-tool] Add chip-tool interactive mode * Update generated content * Allow interactive mode to be disabled for some build variants
Problem
chip-tool
does not allow to set up multiple subscriptions.It uses
readline
under the hood which maps tolibedit
on Mac.Change overview
interactive
mode tochip-tool
that can be started usingchip-tool interactive start
and it can be cleanly exited by typingquit()
subscriptions shutdown-subscription
andsubscritions shutdown-subscriptions
commands tochip-tool
wait
argument ofsubscribe
attributes since this is now replaced by the interactiveTesting
I have done various tests locally to check that it works without breaking the current behaviour.