-
Notifications
You must be signed in to change notification settings - Fork 452
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
Bring back pipe backend #354
Comments
Something like this would be better:
|
A new option ( |
I've added this feature to librespot-org, should be easy to adapt: librespot-org/librespot#422 |
Being "easy to adapt" isn't a good reason to implement a feature. The proposal for a The pipe backend on the other hand would already allow, when re-integrated, in two ways
Is there an use case where the |
This is handled by init system.
You'll need to modify systemd unit to use pipe backend. When stdout consumer crashes, it won't be restarted. With subprocess you'll only need to edit spotifyd's config and consumer restarts will be handled automatically. |
Let me rephrase: If spotifyd fails, then both processes have to be restarted. It doesn't matter if such restart is automatic, there's still the instability in one process taking down the other. It doesn't need to be case with the
The subcommand approach requires the single systemd unit to have maximum permissions. With the pipe backend, you can have one system unit, running a separate user, minimal permissions, only one directory and network. Then the playback is either a regular process, or an user systemd service. In sum, the subcommand backend doesn't add any new capability over the pipe backend, which allows more control over the processes. |
Is there anything I can do, to get pipe backend ? Just to get you right: |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
add pipe as a backend, which sends audio to stdout or the file handle given to --device Fixes #354
#283 removed the pipe backend during the Great Refactoring (TM pending). Might be a good idea to bring it back at some point.
Thinks to take care of:
--backend pipe
is specified and the daemon runs with the--no-daemon
flag, all logging would have to happen on stderr instead.Backend
enum.setup.rs
(I think this is were the backend logic lies)."Fixes" #256 by basically providing the functionality in the first place.
The text was updated successfully, but these errors were encountered: