On 12/19/22 12:23, Fiona Ebner wrote: > Am 19.12.22 um 12:14 schrieb Stefan Sterz: >> when a vm is configured to use the physical cd rom drive but there is >> no such drive a cryptic "uninitialized value" error is thrown. this >> is due to `$path` not being defined in `sub >> print_drive_commandline_full` in this case. warn that no cd rom drive >> is available and default back to using "none" as media instead. >> >> note that the error was basically cosmetic as the vm would start just >> fine. >> >> forum thread: https://forum.proxmox.com/threads/119592/ >> >> Signed-off-by: Stefan Sterz <s.st...@proxmox.com> >> --- >> changes from v1: make the error message more concise. use shorter >> forum link for better formatting in commit message (thanks @ >> Fiona Ebner). >> >> PVE/QemuServer.pm | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm >> index dd6ea3e..d2f2094 100644 >> --- a/PVE/QemuServer.pm >> +++ b/PVE/QemuServer.pm >> @@ -1292,6 +1292,9 @@ sub get_cdrom_path { >> return $cdrom_path = "/dev/cdrom" if -l "/dev/cdrom"; >> return $cdrom_path = "/dev/cdrom1" if -l "/dev/cdrom1"; >> return $cdrom_path = "/dev/cdrom2" if -l "/dev/cdrom2"; >> + >> + warn "no physical cdrom available, defaulting back to 'none'"; > > It's still having the very same issues as in v1. > >> + return ''; >> } >> >> sub get_iso_path {
yeah sorry, missed that, just send a mail. disregard this. sorry. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
