Skip to content

Conversation

@stormdragon2976
Copy link

This adds Orca support. It falls back gracefully to speech-dispatcher if there is a problem, e.g. the Orca version isn't new enough, or something is up with dbus.

@ctoth
Copy link
Collaborator

ctoth commented Jun 16, 2025

Thanks for your work on this!

@ctoth
Copy link
Collaborator

ctoth commented Jun 16, 2025

Hey, I'm curious about the implementation approach here - I noticed this uses subprocess calls to gdbus rather than a Python D-Bus binding like dbus-python or pydbus. Was there a specific reason for going with the subprocess approach? Just wondering about the tradeoffs considered (dependencies, compatibility, etc.).

Also, I noticed in the silence() method that the path becomes self.main_path + "/Speech" - is that accessing a submodule in Orca's D-Bus interface? Just curious about how that part of the API is structured.

The code looks clean and well-documented - just interested in understanding the design choices!

@stormdragon2976
Copy link
Author

stormdragon2976 commented Jun 16, 2025 via email

@stormdragon2976
Copy link
Author

stormdragon2976 commented Jun 18, 2025 via email

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants