On Fri, Jun 03, 2016 at 10:46:03PM +0100, Richard W.M. Jones wrote:
> On Fri, Jun 03, 2016 at 10:36:52PM +0100, Richard W.M. Jones wrote:
> > On Fri, Jun 03, 2016 at 08:23:01PM +0300, Roman Kagan wrote:
> > > Some virtio-win drivers contain more files than just *.{cat,inf,sys}.
> > > They are filtered out currently, which prevents the drivers from being
> > > installed by PnP.
> > > 
> > > Stop filtering driver files by extension, and copy all of them instead.
> > > 
> > > Signed-off-by: Roman Kagan <[email protected]>
> > > ---
> > >  v2v/windows_virtio.ml | 11 -----------
> > >  1 file changed, 11 deletions(-)
> > > 
> > > diff --git a/v2v/windows_virtio.ml b/v2v/windows_virtio.ml
> > > index 6c8396c..354aa19 100644
> > > --- a/v2v/windows_virtio.ml
> > > +++ b/v2v/windows_virtio.ml
> > > @@ -291,17 +291,6 @@ and virtio_iso_path_matches_guest_os path inspect =
> > >       * elements.
> > >       *)
> > >      let lc_path = String.lowercase_ascii path in
> > > -    let lc_basename = Filename.basename lc_path in
> > > -
> > > -    let extension =
> > > -      match last_part_of lc_basename '.' with
> > > -      | Some x -> x
> > > -      | None -> raise Not_found
> > > -    in
> > > -
> > > -    (* Skip files without specific extensions. *)
> > > -    let extensions = ["cat"; "inf"; "pdb"; "sys"] in
> > > -    if not (List.mem extension extensions) then raise Not_found;
> > >  
> > >      (* Using the full path, work out what version of Windows
> > >       * this driver is for.  Paths can be things like:
> > 
> > At a guess this will break the test suite.  I can't check right now
> > because I'm running another test on my laptop, but will try it later.
> 
> Yes, this breaks v2v_unit_tests.

Indeed.  Turns out I didn't have ocaml-ounit installed so didn't notice.

Roman.

_______________________________________________
Libguestfs mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libguestfs

Reply via email to