all lists on 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal