From: Gilles Pietri <contact+dev@gilouweb.com>
To: pve-devel@lists.proxmox.com
Subject: Re: [pve-devel] Audio support, dummy/none
Date: Tue, 5 Jan 2021 20:33:11 +0100 [thread overview]
Message-ID: <6d7faae2-696c-2419-e946-dc5047a3bd96@gilouweb.com> (raw)
In-Reply-To: <819af18e-a533-013e-023a-6105de644eaa@proxmox.com>
Le 05/01/2021 à 12:43, Aaron Lauterer a écrit :
> Adding support for the 'none' backend should not be too hard to
> implement. Besides changes in QemuServer.pm, the GUI (pve-manager) needs
> to be adapted as well because right now the backend is hard coded.
I'll start with that, it's a good "non intrusive" change that can get me
more acquainted with the source and the overall build system, and
separate git repos.
>
> Adding support for ALSA and / or pulseaudio is probably a bit more
> complicated to get it working "out of the box" and AFAICT our qemu build
> currently does not support the PA backend.
True, and it will also require something like a second layer of
dependent options, so it might make sense to have that stored/offered
somewhere only once for API/qm and the Web UI, I'll think about it though…
>
>
> If you want to take the challenge to implement the support for the none
> backend yourself, you are welcome to do so!
> Have a look at our Dev documentation [0]. We would also need a signed
> CLA[1] send to office@proxmox.com
I just did that, I like the idea of starting with a small feature ;)
>
> Alternatively you can create an enhancement request in our bugtracker so
> we can keep track of it.
I'll probably do it for alsa/pulseaudio support then, as this might be a
little bit more delicate, and might require discussion.
Thanks for your assistance and patience, hopefully I'll be able to help
that way ;)
Regards,
Gilou
prev parent reply other threads:[~2021-01-05 19:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-03 0:55 Gilles Pietri
2021-01-04 15:55 ` Aaron Lauterer
2021-01-04 16:22 ` Gilles Pietri
2021-01-04 17:55 ` Gilles Pietri
2021-01-05 11:43 ` Aaron Lauterer
2021-01-05 19:33 ` Gilles Pietri [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6d7faae2-696c-2419-e946-dc5047a3bd96@gilouweb.com \
--to=contact+dev@gilouweb.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox