> Hi Michael,
> 
> the latest code from svn doesn't work here with lenny. I needed to change
> this in order to get a installation because %vg_info is undefined:
> 
> --- Volumes.pm  2009-06-16 20:39:34.000000000 +0200
> +++ Volumes.pm.new      2009-06-16 21:07:55.000000000 +0200
> @@ -308,9 +308,13 @@
> 
> # store the vg size in MB
> my %vg_info = get_volume_group_information($vg);
> -    $FAI::current_lvm_config{$vg}{size} =
> -      &FAI::convert_unit( $vg_info{alloc_pe_size} .
> -        $vg_info{alloc_pe_size_unit} );
> +    if (defined %vg_info) {
> +      $FAI::current_lvm_config{$vg}{size} =
> +        &FAI::convert_unit( $vg_info{alloc_pe_size} .
> +          $vg_info{alloc_pe_size_unit} );
> +    } else {
> +      $FAI::current_lvm_config{$vg}{size} = "0MB";
> +    }
> 
> # store the logical volumes and their sizes
> my %lv_info = get_logical_volume_information($vg);
> 

The patch is now integrated in 3.2.21+experimental9 (with a slight change as
suggested by perl compiler: defined %vg_info changed to simple %vg_info).

Thanks for testing and suggesting the patch!

Best,
Michael

Attachment: pgpvXpMay529C.pgp
Description: PGP signature

Reply via email to