[PATCH v6 3/4] remove sysconfig files
Daniel P. Berrangé
berrange at redhat.com
Tue Jan 11 09:56:30 UTC 2022
On Tue, Jan 11, 2022 at 09:51:05AM +0000, Andrea Bolognani wrote:
> On Tue, Jan 11, 2022 at 10:21:32AM +0100, Olaf Hering wrote:
> > Mon, 3 Jan 2022 03:18:11 -0800 Andrea Bolognani <abologna at redhat.com>:
> > > The fact that we still QEMU_AUDIO_DRV and SDL_AUDIODRIVER in the
> > > service file even after your changes goes against this principle.
> > So what should be done about these?
> > I think whoever added support for these environment variables a couple
> > of decades ago failed to provide proper documentation. Or perhaps
> > whoever wrote the "sound" section of formatdomain.html.in failed to
> > recognize the existence of these environment variables.
> Dan, do you think it would be okay to simply drop these from the
> service/sysconfig file? IIUC we have a proper way to set them on a
> per-domain level now.
I don't think we need todo anything differentl - Olaf's patches
are fine in this respect already IMHO. Certainly we don't need
to add more documentation about these than already exists as we
don't need to encourage more usage.
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list