public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
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



      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
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal