Re: add emms-volume-sndioctl

[Yoni Rabkin]

Omar Polo <op@omarpolo.com> writes:

> Hello,
>
> Apologies if this isn't the correct mailing list to submit EMMS patches,
> I couldn't find a better place to send this to.
>
> The following patch adds another emms-volume backend for OpenBSD'
> sndioctl(1).  OpenBSD used to use mixerctl(8) to adjust the volume, but
> with the previous release (or the one before?  I don't remember)
> sndioctl(1) was added and mixerctl(8) is something reserved for lower
> level tasks and requires root now IIRC.
>
> Note that I've added the check for the `sndioctl' executable before the
> others: this is needed to prevent emms to automatically choose mixerctl
> (or pactl since it's not unusual to have pulseaudio installed due to
> some dependency.)  Should I add a comment noting this?

I've commited Omar's code. If anyone has OpenBSD handly, please test it.

-- 
   "Cut your own wood and it will warm you twice"