Libguestfs - Jun 2016 - Re: [PATCH v2 2/2] v2v: remove the 'graphicsmodedisabled' entry in ESP BCD

Thanks, I pushed both patches.  Note you will get conflicts when you
pull, since I have folded in some fixes, largely cosmetic, but there
were two more important ones:

(1) v2v_unit_tests: BIOS -> I_BIOS.  Please run the tests!

(2) The code below
> +    let esp_temp_path = g#mkdtemp "/Windows/Temp/ESP_XXXXXX" in
> +
> +    match inspect.i_firmware with
> +    | I_BIOS -> ()
> +    | I_UEFI esp_list ->
> +      List.iter (
> +        fun dev_path ->
> +        g#mount dev_path esp_temp_path;
> +        fix_win_uefi_bcd esp_temp_path;
> +        g#umount esp_temp_path;
> +      ) esp_list;
> +
> +    g#rmdir esp_temp_path;
doesn't do what you think.  The match statement extends all the way
through to the g#rmdir, so g#rmdir will only be executed along the
I_UEFI path.

My solution to this was to move the g#mkdtemp into the I_UEFI path
also, since we only want that directory when dealing with UEFI, not
with BIOS.

Thanks for your contributions,

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW