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

peertube: add vosk plugin #191497

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

peertube: add vosk plugin #191497

wants to merge 10 commits into from

Conversation

sbruder
Copy link
Contributor

@sbruder sbruder commented Sep 16, 2022

Description of changes

This adds support for the PeerTube transcription plugin using the vosk speech recognition service.

To make this work, it adds plugin infrastructure to PeerTube, both for packages and the NixOS module. To ensure it does not break on upgrades, it also includes a NixOS test for testing the plugin management. While the implementation of the plugin management is not as pretty as it should be, we found no other way to do this because of PeerTube limitations (see the commit messages for more details).

It depends on #186917 for the vosk package and #191494 for making PeerTube work after the switch to openssl 3. It currently is rebased to include commits from both branches, so the changes from those PRs are also shown in the diff.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: python 8.has: changelog 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` labels Sep 16, 2022
@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Sep 16, 2022
@ofborg ofborg bot requested review from immae, Izorkin and mohe2015 September 16, 2022 14:10
sbruder and others added 4 commits September 16, 2022 17:06
This also adds a skeleton for the PeerTube plugins and adds the hello
world plugin, which is useful for testing the plugin infrastructure with
a minimal example that does not have complicated dependencies.
This uses PeerTube’s plugin management scripts to install plugins from a
Nix store path instead of the npm registry.

Because they require a populated filesystem and database, it can’t do
the setup before PeerTube’s first start. Therefore, it creates two
separate systemd services. One is dependent on the files required
already existing, which then is started before PeerTube. In the case the
filesystem structure is not initialised yet, a service is started and
polls for the files to exist. After that, it continues with the plugin
installation and imperatively restarts PeerTube to ensure plugin
activation.

Co-authored-by: a-kenji <[email protected]>
@wegank wegank added the 2.status: merge conflict This PR has merge conflicts with the target branch label Mar 20, 2024
@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: python 8.has: changelog 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants