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
Add support for the Speech dispatcher socket #5272
base: main
Are you sure you want to change the base?
Conversation
The potential problem with the new mount options is they seem to not work simultaneously with currently used Older flatpak clients won't know about new option which means you need to wait few years until it become usable. |
They should work, as long as they're implemented correctly (see #5286). At the risk of stating the obvious, Flatpak should never segfault, even if given incompatible options, so if it does then that's a bug. |
Are we still interested in this one? We most likely need that for the https://bugzilla.mozilla.org/show_bug.cgi?id=1638150 . If so, could you @tyrylu rebase the PR that it can be merged? |
This is mostly wrapping |
Not exactly, at least this handles the case the Speech dispatcher socket is moved to a different location than the default. |
yeah but that's relevant for 1% users and most of them will still miss this feature. Even if this lands tomorrow all apps still need to have |
Yeah, likely, they do. So, we're ignoring the users or teaching them about permission overrides, then? And, of course, by forcing usage of the paths, you're exposing implementation details, which the developers now must learn about, even they don't have to care when using Speech dispatcher, as libspeechd abstracts them from it. |
9b38c19
to
cf1f18e
Compare
The point is this change is irrelevant for near future same as gpg-socket is. It could be merged sure, no problem with that yet for current ff users this doesn't change anything and it still need to use filesystem permission for years to come. I would like to set expectation for this change right as there may be false impression that after this lands ff could add |
I know that. But we can at least respond much simply to others if the came across the same issue and are creating a new app and have the right Flatpak version. |
Hi @tyrylu, can you rebase your change please? |
* License along with this library. If not, see <http:https://www.gnu.org/licenses/>. | ||
* | ||
* Authors: | ||
* Alexander Larsson <[email protected]> |
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.
You can put yourself as an author. Same applies to other files. Also the year can be updated.
This commit allows accessing a Flatpak app the socket used for communication with the Speech dispatcher daemon.
981a981
to
0b034d7
Compare
Is there any updates regarding this issue? |
You need to figure out why it doesn't work because solution proposed here works almost identical to filesystem option. |
I am interested myself, yes, the approach of this PR is basically the same. |
This has been a hard problem to troubleshoot and there is not much help online since this seems to be niche edge case. So far, I was able to make the Appimage version of Koodo Reader to work with speech-dispatcher by adding cmd flag argument It's a similar issue with Brave browser. I hope you can point me to the right direction to solve this problem. Thank you! ps. I am running flatpak 1.15.4 on fedora 38 kernel 6.7 |
Try running |
Thank you for your suggestion, unfortunately it didn't work. |
Where did it fail? Any console error messages? |
Nothing insightful, it's just I can't use TTS inside the the application. Here is the log when running the app from the terminal:
The issue here seems to me is more related to the cmd arg Can you kindly, test that Thank you! |
This PR allows accessing a Flatpak app the socket used for communication with the Speech dispatcher daemon. The issue of this socket not being accessible came when testing a possibility of switching Firefox to the Flatpak on Fedora.