This is perfectly valid in VMWare and the VM just boots with an empty drive.  We
used to just skip the whole drive before, but since we changed how we parse
empty cdrom drives this now results in an error and the user not being able to
even dump the XML.  Instead of erroring out, just keep the drive empty.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1903953

v2:
 - Do not report and reset an error, but handle it more nicely.

v1:
 - https://www.redhat.com/archives/libvir-list/2020-December/msg00840.html

Martin Kletzander (5):
  esx: Unindent unnecessary conditional branch
  tests: Use g_autofree in testParseVMXFileName
  vmx: Make virVMXParseFileName return an integer
  vmx: Allow missing cdrom image file in virVMXParseFileName
  vmx: Treat missing cdrom-image as empty drive

 src/esx/esx_driver.c                          | 160 ++++++++++--------
 src/vmware/vmware_conf.c                      |  10 +-
 src/vmx/vmx.c                                 |  25 +--
 src/vmx/vmx.h                                 |   5 +-
 .../vmx2xmldata/vmx2xml-cdrom-ide-missing.vmx |   6 +
 tests/vmx2xmltest.c                           |  36 ++--
 6 files changed, 136 insertions(+), 106 deletions(-)
 create mode 100644 tests/vmx2xmldata/vmx2xml-cdrom-ide-missing.vmx

-- 
2.29.2