diff options
author | Chris Luke <chrisy@flirble.org> | 2017-10-26 10:44:43 -0400 |
---|---|---|
committer | Florin Coras <florin.coras@gmail.com> | 2017-10-27 22:47:16 +0000 |
commit | ae5a02f8235b9a243df09b42e932ae5f238e366b (patch) | |
tree | 708df1a7df2698b720c4cb0d617c1e9d53f56c89 /src/vnet/session/session_test.c | |
parent | ead85e91ce4235e2b4520c181dfcabc7df73e46d (diff) |
Fix for vppctl and interactive commands (VPP-1038)
- Interactive commands like "ping" read extra input from the
input stream.
- In the case of "ping" it is simply a signal to cease the current
operation.
- "vppctl", in non-interactive mode, will issue a "quit" immediately
after the requested command to queue up closing of the session.
- This resulted in "ping" thinking a keypress was seen and returning
control to the CLI; the "quit" command however is consumed by the
keypress event handler and thus the session does not close.
- This patch reworks vppctl slightly to only issue "quit" after the
command has completed. In particular it uses the fact that VPP issues
NUL bytes as a surrogate prompt between output of commands to signal
acknowledgement that the command has completed; vppctl now flags
that the quit should be issued after the next such acknowledgement.
- Since input it still accepted, the user can still terminate the
"ping" early, if desired.
Change-Id: I7e3dbe767f32f8e364ccb5f81799759b311585df
Signed-off-by: Chris Luke <chrisy@flirble.org>
Diffstat (limited to 'src/vnet/session/session_test.c')
0 files changed, 0 insertions, 0 deletions