[pve-devel] [PATCH qemu-server v16 4/12] control: add virtiofsd as runtime dependency for qemu-server

[Thomas Lamprecht]

Am 04.04.25 um 18:08 schrieb Markus Frank:
> Signed-off-by: Markus Frank <m.frank at proxmox.com>
> Reviewed-by: Fiona Ebner <f.ebner at proxmox.com>
> Reviewed-by: Laurențiu Leahu-Vlăducu <l.leahu-vladucu at proxmox.com
> Reviewed-by: Daniel Kral <d.kral at proxmox.com>
> Tested-by: Laurențiu Leahu-Vlăducu <l.leahu-vladucu at proxmox.com>
> Tested-by: Daniel Kral <d.kral at proxmox.com>
> Tested-by: Lukas Wagner <l.wagner at proxmox.com>
> ---
> no changes in v16
> 
>  debian/control | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/debian/control b/debian/control
> index 196647dd..27925009 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -55,6 +55,7 @@ Depends: dbus,
>           socat,
>           swtpm,
>           swtpm-tools,
> +         virtiofsd,
>           ${misc:Depends},
>           ${perl:Depends},
>           ${shlibs:Depends},

What about adding this as recommends? It would be still pulled in by
default, user can opt out though, reducing the attack surface.

Would your code handle the case where the executable does not exist
with a somewhat nice error?