[MSI] setup should not enumerate read-only CDRoms as install-targets CORE-14432
authorJoachim Henze <Joachim.Henze@reactos.org>
Sat, 5 Jun 2021 19:27:01 +0000 (21:27 +0200)
committerJoachim Henze <Joachim.Henze@reactos.org>
Sat, 5 Jun 2021 19:27:40 +0000 (21:27 +0200)
commit864e20b881130cbc9668b8e253a31b8c325e9b99
treefb07a148e63108616a20f2e5ae43fc7f8768d985
parent83fcd65700e8cf6d0c07c81d85a07e68c5d87cdc
[MSI] setup should not enumerate read-only CDRoms as install-targets CORE-14432

We decided to check a bit different than Wine currently does.
Wine checks for
if (GetVolumeInformationW(ptr, NULL, 0, NULL, 0, &flags, NULL, 0) && flags & FILE_READ_ONLY_VOLUME)
to exclude read-only-media while we (for now) check for
if (GetDriveTypeW(ptr) == DRIVE_CDROM)

For now this gives us the benefit of covering also CDROM drives, that do not have
a CD put into it.
That does not work properly for us with the Wine patch yet, but it does for Wine!
So this maybe points to some other bug in ros, maybe in our CDROM(FS)-drivers?

In the future we should also test this on other read-only-media in Windows e.g.
a read-only USB stick to decide whether we want to stick with our current solution
or whether we want to switch to Wines solution or whether we want to logically OR
both conditions!

Thanks to Doug Lyons for providing what *currently* does work best for us!

VBox https://reactos.org/testman/compare.php?ids=77537,77546 (no change)
KVM https://reactos.org/testman/compare.php?ids=77531,77541 (no change)
dll/win32/msi/dialog.c