On Tue, Oct 27, 2015 at 09:12:41AM +0000, Richard W.M. Jones wrote: > On Mon, Oct 26, 2015 at 09:00:03PM +0300, Roman Kagan wrote: > > Windows QXL drivers include also qxldd.dll which used to get filtered > > out and not copied over into the guest. As a result QXL driver failed > > to install due to a missing file. > > (* Skip files without specific extensions. *) > > - let extensions = ["cat"; "inf"; "pdb"; "sys"] in > > + let extensions = ["cat"; "dll"; "inf"; "pdb"; "sys"] in > > Actually no, this part is (may be) wrong. > > This causes WdfCoInstaller*.dll from the ISO to be copied in. It's > not clear to me whether these files are needed - I suspect not.
Yes it is, see the corresponding *.inf where it's mentioned in the corresponding CopyFiles directive (and yes, I just verified that with that file missing the balloon driver installation fails with 'file not found' error). As a matter of fact I tend to think that filtering of files by suffix is a bad idea in general: the packager of the drivers should be free to ship any files he wants, including docs and text notes. It's only the path that should match against the guest OS flavor and that really matters. E.g. the balloon driver directory also contains blnsvr.exe (and its .pdb) which is a usermode service responsible for feeding the balloon device with the memory stats; it's not a part of the driver per se but it's currently bundled with it. (I'm yet to figure out who's responsible for installing it). > It also causes the tests to fail, although only when you have > installed the optional ocaml oUnit2 package: That's a problem with the test then (and no I didn't have that package installed so my make -C v2v check passed). Roman. _______________________________________________ Libguestfs mailing list [email protected] https://www.redhat.com/mailman/listinfo/libguestfs
