FS#65872 - [mpd][feature request] add back if possible sndio backend support
Attached to Project:
Arch Linux
Opened by Javier (je-vv) - Tuesday, 17 March 2020, 22:48 GMT
Last edited by Gaetan Bisson (vesath) - Sunday, 26 July 2020, 19:05 GMT
Opened by Javier (je-vv) - Tuesday, 17 March 2020, 22:48 GMT
Last edited by Gaetan Bisson (vesath) - Sunday, 26 July 2020, 19:05 GMT
|
Details
Description: Currently mpd is built with:
> '-Dsndio=disabled' # interferes with detection of alsa devices According to the comment, sndio migth interfere with alsa devices detection, but I guess, if that's still valid (perhaps it's no longer valid), it might happen only if sndio is available in the system. For thouse wanting to try and use sndio as the audio backend, it'll be very useful to remove the disable of sndio support. |
This task depends upon
Closed by Gaetan Bisson (vesath)
Sunday, 26 July 2020, 19:05 GMT
Reason for closing: Won't implement
Additional comments about closing: Cons outweigh pros. See discussion.
Sunday, 26 July 2020, 19:05 GMT
Reason for closing: Won't implement
Additional comments about closing: Cons outweigh pros. See discussion.
So uh, did you even check this? If we take the comment at face value, it is still valid. There is no feature request here if even you don't know whether it is valid to ask for it.
> it might happen only if sndio is available in the system.
If mpd builds against libsndio.so then it is available in the system because that is how libraries work. There is no "optional at runtime" here.
[0] https://git.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/mpd&id=82088afed843750c831692fbdcb5c6ea12bbc80f
audio_output {
type "alsa"
name "ALSA Default"
}
At the very end, with a comment to change that to whatever output required according to the wiki and the /usr/share/doc/mpd/mpdconf.example doc file available. That would have just that effect, if no one touches that file, neither created a user config file. I guess it's by design that mpd developers made sndio be selected 1st... I'm not a meson expert, but perhaps there's also a way to set the default order of output thorugh some build configuration... By modifying conf, mpd would be assuring that default behavior, and if needing something different, like pulse, there's the need to edit that file, or the user config one any ways, right?
And again, the reason for the request is to move from no support at all for sndio, to actually support it...
Thanks !
Yes, more lines can be added to /etc/mpd.conf to reproduce the current behavior, but I do not consider this to be a satisfying solution. First, because we would need to modify every existing /etc/mpd.conf and, second, because omitting that part of the configuration is a feature; mpdconf.example says:
# MPD supports various audio output types, as well as playing through multiple
# audio outputs at the same time, through multiple audio_output settings
# blocks. Setting this block is optional, though the server will only attempt
# autodetection for one sound card.
Since the use case for sndio output does not appear very compelling to me (because it has been dropped for more than a year and nobody noticed), I'm quite reluctant to implement this unless we can find a simple way to autodetect alsa devices by default without changing too much code (and without changing existing /etc/mpd.conf).