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

FEAT(client): Add "starttalking" and "stoptalking" to the socket RPC interface #4754

Merged
merged 2 commits into from
Feb 13, 2021

Conversation

cortiv
Copy link
Contributor

@cortiv cortiv commented Feb 13, 2021

FEAT(client): add starttalking and stoptalking to the socket rpc interface

If mumble isn't allowed to listen to all input from an input device, for example when running as wayland client, ptt events can be sent via socket rpc, and, in the case of wayland, the keybinding can be done by the compositor. #3675 implements this for dbus.
To avoid executing the rather large mumble binary to do the rpc, one could also do something like this:
echo '<self><starttalking>starttalking</starttalking></self>' | socat - UNIX-CONNECT:/tmp/MumbleSocket

Fixes #1594

@davidebeatrici davidebeatrici changed the title Add push-to-talk to socket rpc interface FEAT(client): Add "starttalking" and "stoptalking" to the socket RPC interface Feb 13, 2021
@davidebeatrici
Copy link
Member

Could you add the PR's message to db93e65, please?

I would also set the same title as the PR.

…rface

If mumble isn't allowed to listen to all input from an input device, for example when running as wayland client, ptt events can be sent via socket rpc, and, in the case of wayland, the keybinding can be done by the compositor. mumble-voip#3675 implements this for dbus.
To avoid executing the rather large mumble binary to do the rpc, one could also do something like this:
echo '<self><starttalking>starttalking</starttalking></self>' | socat - UNIX-CONNECT:/tmp/MumbleSocket

Implements mumble-voip#1594
Scanning directory './src'...
Scanning directory './src/mumble'...
Updating 'src/mumble/mumble_en.ts'...
    Found 1954 source text(s) (1 new and 1953 already existing)
    Removed 1 obsolete entries
@cortiv
Copy link
Contributor Author

cortiv commented Feb 13, 2021

Done

@davidebeatrici davidebeatrici merged commit 3d7ce49 into mumble-voip:master Feb 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No Client RPC support for Push-to-Talk?
3 participants