Commit c567ae6
committed
feat(streaming_client): add --rpc-port flag to ola_streaming_client
Closes #1994 for the streaming-client tool. The RPC port has been
tuneable on olad via -r/--rpc_port since forever (olad/OlaServer.cpp:66);
this exposes the same knob on ola_streaming_client so users can point
the CLI at a non-default olad instance without needing to edit config.
The underlying StreamingClient::Options.server_port field already exists
(ola/StreamingClient.cpp:48,58) and flows through to
ConnectToServer(m_server_port). This PR only surfaces it via the CLI:
- New DEFINE_s_uint16(rpc_port, r, ola::OLA_DEFAULT_PORT, ...) matching
olad's flag name, short form, and default (9010 from
include/ola/Constants.h:68).
- Existing 'StreamingClient ola_client;' replaced with an Options
construction that carries FLAGS_rpc_port.
ola_dmxmonitor and ola_dmxconsole use OlaClientWrapper, which does not
yet carry a server_port knob. Extending the wrapper is a separate
change; this PR is intentionally scoped to the one tool where the
library already supports the option.1 parent c6196f7 commit c567ae6
1 file changed
Lines changed: 6 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
47 | 48 | | |
48 | 49 | | |
49 | 50 | | |
| 51 | + | |
| 52 | + | |
50 | 53 | | |
51 | 54 | | |
52 | 55 | | |
| |||
76 | 79 | | |
77 | 80 | | |
78 | 81 | | |
79 | | - | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
80 | 85 | | |
81 | 86 | | |
82 | 87 | | |
| |||
0 commit comments