Andrea,

Applied the patch and it worked perfectly on a couple test systems.

Thanks!

Jonathan


Andrea Righi wrote:
> Jonathan,
> 
> the following patch fixes the problem in my case. I have already checked
> it in the trunk. The fix will be included in the next 3.8.1 release.
> 
> It would be great if you could test it also in your environment...
> 
> Regards,
> -Andrea
> 
> Index: lib/SystemImager/Common.pm
> ===================================================================
> --- lib/SystemImager/Common.pm  (revision 3959)
> +++ lib/SystemImager/Common.pm  (working copy)
> @@ -10,6 +10,7 @@
>  package SystemImager::Common;
> 
>  use strict;
> +use POSIX qw/ceil/;
>  use vars qw($version_number $VERSION);
> 
>  $version_number="SYSTEMIMAGER_VERSION_STRING";
> @@ -537,6 +538,16 @@
>                          $startMB =~ s/(\d+)MB/$1/go;
>                          $endMB   =~ s/(\d+)MB/$1/go;
> 
> +                        # Fix partition size with old versions of parted.
> +                        if (version_cmp($parted_version, '1.6.23') < 0) {
> +                            if ($startMB) {
> +                                $startMB = ceil($startMB * 1024 * 1024 / 
> 1000 / 1000);
> +                            }
> +                            if ($endMB) {
> +                                $endMB = ceil($endMB * 1024 * 1024 / 1000 / 
> 1000);
> +                            }
> +                        }
> +
>                          #
>                          # Get rid of parted's fs info.  We don't use it.  
> But we do need
>                          # 'name' and 'flags', and it's a pain in the but to 
> parse this
> @@ -596,6 +607,16 @@
>                          $startMB =~ s/(\d+)MB/$1/go;
>                          $endMB   =~ s/(\d+)MB/$1/go;
> 
> +                        # Fix partition size with old versions of parted.
> +                        if (version_cmp($parted_version, '1.6.23') < 0) {
> +                            if ($startMB) {
> +                                $startMB = ceil($startMB * 1024 * 1024 / 
> 1000 / 1000);
> +                            }
> +                            if ($endMB) {
> +                                $endMB = ceil($endMB * 1024 * 1024 / 1000 / 
> 1000);
> +                            }
> +                        }
> +
>                          #
>                          # Get rid of parted's fs info.  We don't use it.  
> But we do need
>                          # 'name' and 'flags', and it's a pain in the but to 
> parse this
> 




-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
sisuite-devel mailing list
sisuite-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sisuite-devel

Reply via email to