Skip to content

Conversation

@fpfcmsr
Copy link

@fpfcmsr fpfcmsr commented Dec 23, 2025

Creates file level backups within speech-dispatcher directory.
Tested scenarios:

  • no directory present (must create)
  • directory present and no config files (must create)
  • directory present and config files present (must backup)
  • deleting the config files after generation -> gets recreated when new voice selected

Tested the binary generated by build, did not test snap/flatpak specific builds.

tested on a clean install - Pied looks in .config/speech-dispatcher, creates a backup if it exists, and overwrites it. This makes it clear what permissions pied needs as opposed to the broad filesystem=home.
Allows pied to create speech-dispatcher directory if not present, and gives write options to said directory.
Creates backup of speechd.conf and piper.conf individually within speech-dispatcher directory. If either is changed / removed, then recreates them / backs them up as needed.
@Elleo
Copy link
Owner

Elleo commented Jan 24, 2026

Hi @fpfcmsr,

Just wanted to add a quick note just to let you know that I haven't forgotten about this. I have given the code a brief review, and it looks good (there is a typo in the config directory in the flatpak branch, but I'll fix that when I merge it.) Unfortuantely my laptop died recently, and while I've got mostly up and running with an old one, I've run into some GStreamer linking issues building on the latest version of Ubuntu, so I haven't been able to give it a full test yet.

@fpfcmsr
Copy link
Author

fpfcmsr commented Jan 30, 2026

Hi @fpfcmsr,

Just wanted to add a quick note just to let you know that I haven't forgotten about this. I have given the code a brief review, and it looks good (there is a typo in the config directory in the flatpak branch, but I'll fix that when I merge it.) Unfortuantely my laptop died recently, and while I've got mostly up and running with an old one, I've run into some GStreamer linking issues building on the latest version of Ubuntu, so I haven't been able to give it a full test yet.

Thank you for the update! No worries, take your time. Let me know if there's anything I can do on the testing end and hope things are back on the upward track.

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