]> git.proxmox.com Git - pve-installer.git/blobdiff - proxinstall
stop main loop when closing the main window
[pve-installer.git] / proxinstall
index 5ea3692c30f52f3861991c68c2c1b7fa88ac35b4..21c7201eb388430cf957cc99b99346a8fad4c04e 100755 (executable)
@@ -19,9 +19,9 @@ use File::Path;
 use Time::HiRes;
 use POSIX ":sys_wait_h";
 
-use Proxmox::Install::Env;
+use Proxmox::Install::ISOEnv;
 use Proxmox::Log;
-use Proxmox::Sys::Block qw(get_cached_disks);
+use Proxmox::Sys::Block qw(get_cached_disks wipe_disk partition_bootable_disk);
 use Proxmox::Sys::Command qw(run_command syscmd);
 use Proxmox::Sys::File qw(file_read_firstline file_read_all file_write_all);
 use Proxmox::Sys::Net qw(parse_ip_address parse_ip_mask);
@@ -36,12 +36,12 @@ if (!$ENV{G_SLICE} ||  $ENV{G_SLICE} ne "always-malloc") {
        'test-image|t=s' => \$test_image
     ) or die "usage error\n";
 
-    Proxmox::Install::Env::set_test_image($test_image) if $test_image;
+    Proxmox::Install::ISOEnv::set_test_image($test_image) if $test_image;
 }
 
 $ENV{'LVM_SUPPRESS_FD_WARNINGS'} = '1';
 
-my $env = Proxmox::Install::Env::setup();
+my $env = Proxmox::Install::ISOEnv::setup();
 
 my $zfstestpool = "test_rpool";
 my $zfspoolname = is_test_mode() ? $zfstestpool : 'rpool';
@@ -155,7 +155,6 @@ my $timezone = 'Europe/Vienna';
 my $keymap = 'en-us';
 my $password;
 my $mailto = 'mail@example.invalid';
-my $cmap = $env->{locales};
 my $autoreboot_seconds = 5;
 
 my $config = {
@@ -236,10 +235,21 @@ compatibility_level = 2
 _EOD
 
 
-sub detect_country {
+sub app_quit {
+    my ($exit_code) = @_;
+
+    Gtk3->main_quit() if Gtk3->main_level() > 0;
+
+    # reap left over zombie processes
+    while ((my $child = waitpid(-1, POSIX::WNOHANG)) > 0) {
+       print "reaped child $child\n";
+    }
+    exit($exit_code);
+}
 
+sub detect_country {
     print "trying to detect country...\n";
-    my $cpid = open2(\*TMP, undef, "traceroute -N 1 -q 1 -n 8.8.8.8");
+    my $cpid = open2(my $TRACEROUTE_FH, undef, "traceroute -N 1 -q 1 -n 8.8.8.8");
     return undef if !$cpid;
 
     my $country;
@@ -248,7 +258,7 @@ sub detect_country {
     eval  {
        local $SIG{ALRM} = sub { die "timed out!\n" };
        my $line;
-       while (defined ($line = <TMP>)) {
+       while (defined ($line = <$TRACEROUTE_FH>)) {
            log_debug("DC TRACEROUTE: $line");
            if ($line =~ m/\s*\d+\s+(\d+\.\d+\.\d+\.\d+)\s/) {
                my $geoip = `geoiplookup $1`;
@@ -261,12 +271,10 @@ sub detect_country {
            }
        }
     };
-
     my $err = $@;
-
     alarm ($previous_alarm);
 
-    close (TMP);
+    close($TRACEROUTE_FH);
 
     if ($err) {
        print "unable to detect country - $err\n";
@@ -495,7 +503,7 @@ sub ask_existing_vg_rename_or_abort {
                die "could not rename VG from '$vg->{pvs}' ($vg_uuid) to '$new_vgname'!\n";
        }
     } else {
-       set_next("_Reboot", sub { exit (0); } );
+       set_next("_Reboot", sub { app_quit(0); } );
        display_html("fail.htm");
        die "Cancled installation by user, due to already existing volume group '$vgname'\n";
     }
@@ -706,12 +714,8 @@ sub extract_data {
 
     my $bootdevinfo = [];
 
-    my $swapfile;
-    my $rootdev;
-    my $datadev;
-
-    my $use_zfs = 0;
-    my $use_btrfs = 0;
+    my ($swapfile, $rootdev, $datadev);
+    my ($use_zfs, $use_btrfs) = (0, 0);
 
     my $filesys = $config_options->{filesys};
 
@@ -746,7 +750,7 @@ sub extract_data {
 
        if (is_test_mode()) {
 
-           my $test_images = Proxmox::Install::Env::get_test_images();
+           my $test_images = Proxmox::Install::ISOEnv::get_test_images();
            $rootdev = abs_path($test_images->[0]); # FIXME: use all selected for test too!
            syscmd("umount $rootdev");
 
@@ -862,7 +866,7 @@ sub extract_data {
 
            update_progress(0, 0.02, $maxper, "create partitions");
 
-           my $logical_bsize = logical_blocksize($target_hd);
+           my $logical_bsize = Proxmox::Sys::Block::logical_blocksize($target_hd);
 
            my ($os_size, $osdev, $efidev) =
                partition_bootable_disk($target_hd, $config_options->{hdsize}, '8E00');
@@ -1126,7 +1130,7 @@ sub extract_data {
        }
 
        # Note: keyboard-configuration/xbkb-keymap is used by console-setup
-       my $xkmap = $cmap->{kmap}->{$keymap}->{x11} // 'us';
+       my $xkmap = $env->{locales}->{kmap}->{$keymap}->{x11} // 'us';
 
        debconfig_set ($targetdir, <<_EOD);
 locales locales/default_environment_locale select en_US.UTF-8
@@ -1188,7 +1192,7 @@ _EOD
        file_write_all("$targetdir/etc/timezone", "$timezone\n");
 
        # set apt mirror
-       if (my $mirror = $cmap->{country}->{$country}->{mirror}) {
+       if (my $mirror = $env->{locales}->{country}->{$country}->{mirror}) {
            my $fn = "$targetdir/etc/apt/sources.list";
            syscmd("sed -i 's/ftp\\.debian\\.org/$mirror/' '$fn'");
        }
@@ -1269,13 +1273,11 @@ _EOD
                            push @$bootloader_err_list, $@ if $@;
                        }
 
-                   eval {
                        if (my $esp = $di->{esp}) {
-                               prepare_grub_efi_boot_esp($dev, $esp, $targetdir);
-                           }
+                           eval { prepare_grub_efi_boot_esp($dev, $esp, $targetdir) };
+                           push @$bootloader_err_list, $@ if $@;
                        }
-                   };
-                   push @$bootloader_err_list, $@ if $@;
+                   }
                }
 
                syscmd("chroot $targetdir /usr/sbin/update-grub") == 0 ||
@@ -1315,7 +1317,7 @@ _EOD
            mkdir $tmpdir;
 
            # write vnc keymap to datacenter.cfg
-           my $vnckmap = $cmap->{kmap}->{$keymap}->{kvm} || 'en-us';
+           my $vnckmap = $env->{locales}->{kmap}->{$keymap}->{kvm} || 'en-us';
            file_write_all("$tmpdir/datacenter.cfg", "keyboard: $vnckmap\n");
 
            # save admin email
@@ -1496,11 +1498,12 @@ sub create_main_window {
 
     $window = Gtk3::Window->new();
     $window->set_default_size(1024, 768);
-    $window->set_has_resize_grip(0);
+    $window->signal_connect(map => sub { $window->set_resizable(0); });
     $window->fullscreen() if !is_test_mode();
     $window->set_decorated(0) if !is_test_mode();
+    $window->signal_connect(destroy => sub { Gtk3->main_quit(); });
 
-    my $vbox = Gtk3::VBox->new(0, 0);
+    my $vbox = Gtk3::Box->new('vertical', 0);
 
     my $logofn = "$env->{product}-banner.png";
     my $image = Gtk3::Image->new_from_file("${proxmox_libdir}/$logofn");
@@ -1513,17 +1516,17 @@ sub create_main_window {
 
     $vbox->pack_start($image, 0, 0, 0);
 
-    my $hbox = Gtk3::HBox->new(0, 0);
+    my $hbox = Gtk3::Box->new('horizontal', 0);
     $vbox->pack_start($hbox, 1, 1, 0);
 
  #  my $f1 = Gtk3::Frame->new ('test');
  #  $f1->set_shadow_type ('none');
  #  $hbox->pack_start ($f1, 1, 1, 0);
 
-    my $sep1 = Gtk3::HSeparator->new();
+    my $sep1 = Gtk3::Separator->new('horizontal');
     $vbox->pack_start($sep1, 0, 0, 0);
 
-    $cmdbox = Gtk3::HBox->new();
+    $cmdbox = Gtk3::Box->new('horizontal', 0);
     $vbox->pack_start($cmdbox, 0, 0, 10);
 
     $next = Gtk3::Button->new('_Next');
@@ -1539,33 +1542,34 @@ sub create_main_window {
     my $abort = Gtk3::Button->new('_Abort');
     $abort->set_can_focus(0);
     $cmdbox->pack_start($abort, 0, 0, 10);
-    $abort->signal_connect(clicked => sub { exit (-1); });
+    $abort->signal_connect(clicked => sub { app_quit(-1); });
 
-    my $vbox2 = Gtk3::VBox->new(0, 0);
+    my $vbox2 = Gtk3::Box->new('vertical', 0);
     $hbox->add($vbox2);
 
     $htmlview = Gtk3::WebKit2::WebView->new();
+    $htmlview->set_hexpand(1);
     my $scrolls = Gtk3::ScrolledWindow->new();
     $scrolls->add($htmlview);
 
-    my $hbox2 = Gtk3::HBox->new(0, 0);
+    my $hbox2 = Gtk3::Box->new('horizontal', 0);
     $hbox2->pack_start($scrolls, 1, 1, 0);
 
     $vbox2->pack_start($hbox2, 1, 1, 0);
 
-    my $vbox3 = Gtk3::VBox->new(0, 0);
+    my $vbox3 = Gtk3::Box->new('vertical', 0);
     $vbox2->pack_start($vbox3, 0, 0, 0);
 
-    my $sep2 = Gtk3::HSeparator->new;
+    my $sep2 = Gtk3::Separator->new('horizontal');
     $vbox3->pack_start($sep2, 0, 0, 0);
 
-    $inbox = Gtk3::HBox->new(0, 0);
+    $inbox = Gtk3::Box->new('horizontal', 0);
     $vbox3->pack_start($inbox, 0, 0, 0);
 
     $window->add($vbox);
 
     $window->show_all;
-    $window->realize();
+    $window->present();
 }
 
 sub cleanup_view {
@@ -1619,7 +1623,7 @@ sub create_text_input {
 
     my $label = Gtk3::Label->new($text);
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox->pack_start($label, 0, 0, 10);
     my $e1 = Gtk3::Entry->new();
     $e1->set_width_chars(35);
@@ -1635,7 +1639,7 @@ sub create_cidr_inputs {
 
     my $label = Gtk3::Label->new('IP Address (CIDR)');
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox->pack_start($label, 0, 0, 10);
 
     my $ip_el = Gtk3::Entry->new();
@@ -1645,7 +1649,6 @@ sub create_cidr_inputs {
 
     $label = Gtk3::Label->new('/');
     $label->set_size_request(10, -1);
-    $label->set_alignment(0.5, 0.5);
     $hbox->pack_start($label, 0, 0, 2);
 
     my $cidr_el = Gtk3::Entry->new();
@@ -1749,10 +1752,10 @@ sub create_ipconf_view {
        $device_cb->set_active(0);
     }
 
-    my $devicebox = Gtk3::HBox->new(0, 0);
+    my $devicebox = Gtk3::Box->new('horizontal', 0);
     my $label = Gtk3::Label->new("Management Interface:");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $devicebox->pack_start($label, 0, 0, 10);
     $devicebox->pack_start($device_cb, 0, 0, 0);
 
@@ -1869,7 +1872,7 @@ sub create_ack_view {
 
     cleanup_view();
 
-    my $vbox =  Gtk3::VBox->new(0, 0);
+    my $vbox =  Gtk3::Box->new('vertical', 0);
     $inbox->pack_start($vbox, 1, 0, 0);
 
     my $reboot_checkbox = Gtk3::CheckButton->new('Automatically reboot after successful installation');
@@ -1887,7 +1890,7 @@ sub create_ack_view {
     my %config_values = (
        __target_hd__ => join(' | ', @{$config_options->{target_hds}}),
        __target_fs__ => $config_options->{filesys},
-       __country__ => $cmap->{country}->{$country}->{name},
+       __country__ => $env->{locales}->{country}->{$country}->{name},
        __timezone__ => $timezone,
        __keymap__ => $keymap,
        __mailto__ => $mailto,
@@ -1952,7 +1955,7 @@ sub update_layout {
     my $ind;
     my $def;
     my $i = 0;
-    my $kmaphash = $cmap->{kmaphash};
+    my $kmaphash = $env->{locales}->{kmaphash};
     foreach my $layout (sort keys %$kmaphash) {
        $def = $i if $kmaphash->{$layout} eq 'en-us';
        $ind = $i if $kmap && $kmaphash->{$layout} eq $kmap;
@@ -1973,34 +1976,24 @@ my $lastzonecb;
 sub update_zonelist {
     my ($box, $cc) = @_;
 
-    my $cczones = $cmap->{cczones};
-    my $zones = $cmap->{zones};
-
-    my $sel;
+    my $sel = $timezone; # initial default
     if ($lastzonecb) {
        $sel = $lastzonecb->get_active_text();
-       $box->remove ($lastzonecb);
-    } else {
-       $sel = $timezone; # used once to select default
+       $box->remove($lastzonecb);
     }
 
     my $cb = $lastzonecb = Gtk3::ComboBoxText->new();
     $cb->set_size_request(200, -1);
-
     $cb->signal_connect('changed' => sub {
        $timezone = $cb->get_active_text();
     });
 
-    my @za;
-    if ($cc && defined ($cczones->{$cc})) {
-       @za = keys %{$cczones->{$cc}};
-    } else {
-       @za = keys %$zones;
-    }
-    my $ind;
-    my $i = 0;
-    foreach my $zone (sort @za) {
-       $ind = $i if $sel && $zone eq $sel;
+    my ($cczones, $zones) = $env->{locales}->@{'cczones', 'zones'};
+    my @available_zones = $cc && defined($cczones->{$cc}) ? keys %{$cczones->{$cc}} : keys %$zones;
+
+    my ($i, $selected_index) = (0, undef);
+    for my $zone (sort @available_zones) {
+       $selected_index = $i if $sel && $zone eq $sel;
        $cb->append_text($zone);
        $i++;
     }
@@ -2008,7 +2001,7 @@ sub update_zonelist {
     # Append UTC here, so it is always the last item and never the default for any country.
     $cb->append_text('UTC');
 
-    $cb->set_active($ind || 0);
+    $cb->set_active($selected_index || 0);
 
     $cb->show;
     $box->pack_start($cb, 0, 0, 0);
@@ -2018,15 +2011,15 @@ sub create_password_view {
 
     cleanup_view();
 
-    my $vbox2 =  Gtk3::VBox->new(0, 0);
+    my $vbox2 =  Gtk3::Box->new('vertical', 0);
     $inbox->pack_start($vbox2, 1, 0, 0);
-    my $vbox =  Gtk3::VBox->new(0, 0);
+    my $vbox =  Gtk3::Box->new('vertical', 0);
     $vbox2->pack_start($vbox, 0, 0, 10);
 
-    my $hbox1 = Gtk3::HBox->new(0, 0);
+    my $hbox1 = Gtk3::Box->new('horizontal', 0);
     my $label = Gtk3::Label->new("Password");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox1->pack_start($label, 0, 0, 10);
     my $pwe1 = Gtk3::Entry->new();
     $pwe1->set_visibility(0);
@@ -2034,10 +2027,10 @@ sub create_password_view {
     $pwe1->set_size_request(200, -1);
     $hbox1->pack_start($pwe1, 0, 0, 0);
 
-    my $hbox2 = Gtk3::HBox->new(0, 0);
+    my $hbox2 = Gtk3::Box->new('horizontal', 0);
     $label = Gtk3::Label->new("Confirm");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox2->pack_start($label, 0, 0, 10);
     my $pwe2 = Gtk3::Entry->new();
     $pwe2->set_visibility(0);
@@ -2045,10 +2038,10 @@ sub create_password_view {
     $pwe2->set_size_request(200, -1);
     $hbox2->pack_start($pwe2, 0, 0, 0);
 
-    my $hbox3 = Gtk3::HBox->new(0, 0);
+    my $hbox3 = Gtk3::Box->new('horizontal', 0);
     $label = Gtk3::Label->new("Email");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox3->pack_start($label, 0, 0, 10);
     my $eme = Gtk3::Entry->new();
     $eme->set_size_request(200, -1);
@@ -2111,12 +2104,11 @@ sub create_country_view {
 
     cleanup_view();
 
-    my $countryhash = $cmap->{countryhash};
-    my $ctr = $cmap->{country};
+    my $locales = $env->{locales};
 
-    my $vbox2 =  Gtk3::VBox->new(0, 0);
+    my $vbox2 =  Gtk3::Box->new('vertical', 0);
     $inbox->pack_start($vbox2, 1, 0, 0);
-    my $vbox =  Gtk3::VBox->new(0, 0);
+    my $vbox =  Gtk3::Box->new('vertical', 0);
     $vbox2->pack_start($vbox, 0, 0, 10);
 
     my $w = Gtk3::Entry->new();
@@ -2128,23 +2120,23 @@ sub create_country_view {
     $c->set_popup_set_width(1);
     $c->set_inline_completion(1);
 
-    my $hbox2 = Gtk3::HBox->new(0, 0);
+    my $hbox2 = Gtk3::Box->new('horizontal', 0);
     my $label = Gtk3::Label->new("Time zone");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox2->pack_start($label, 0, 0, 10);
     update_zonelist ($hbox2);
 
-    my $hbox3 = Gtk3::HBox->new(0, 0);
+    my $hbox3 = Gtk3::Box->new('horizontal', 0);
     $label = Gtk3::Label->new("Keyboard Layout");
     $label->set_size_request(150, -1);
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $hbox3->pack_start($label, 0, 0, 10);
 
     my $kmapcb = Gtk3::ComboBoxText->new();
     $kmapcb->set_size_request (200, -1);
-    foreach my $layout (sort keys %{$cmap->{kmaphash}}) {
-       $kmapcb->append_text ($layout);
+    for my $layout (sort keys %{$locales->{kmaphash}}) {
+       $kmapcb->append_text($layout);
     }
 
     update_layout($kmapcb);
@@ -2153,9 +2145,9 @@ sub create_country_view {
     $kmapcb->signal_connect ('changed' => sub {
        my $sel = $kmapcb->get_active_text();
        $last_layout = $kmapcb->get_active();
-       if (my $kmap = $cmap->{kmaphash}->{$sel}) {
-           my $xkmap = $cmap->{kmap}->{$kmap}->{x11};
-           my $xvar = $cmap->{kmap}->{$kmap}->{x11var};
+       if (my $kmap = $locales->{kmaphash}->{$sel}) {
+           my $xkmap = $locales->{kmap}->{$kmap}->{x11};
+           my $xvar = $locales->{kmap}->{$kmap}->{x11var};
            $keymap = $kmap;
 
            return if (defined($installer_kmap) && $installer_kmap eq $kmap);
@@ -2184,9 +2176,9 @@ sub create_country_view {
        my ($entry, $event) = @_;
        my $text = $entry->get_text;
 
-       if (my $cc = $countryhash->{lc($text)}) {
+       if (my $cc = $locales->{countryhash}->{lc($text)}) {
            update_zonelist($hbox2, $cc);
-           my $kmap = $ctr->{$cc}->{kmap} || 'en-us';
+           my $kmap = $locales->{country}->{$cc}->{kmap} || 'en-us';
            update_layout($kmapcb, $kmap);
        }
     });
@@ -2198,20 +2190,19 @@ sub create_country_view {
        my $val = $event->get_keyval;
 
        if ($val == Gtk3::Gdk::KEY_Tab) {
-           my $cc = $countryhash->{lc($text)};
+           my $cc = $locales->{countryhash}->{lc($text)};
 
            my $found = 0;
            my $compl;
 
            if ($cc) {
                $found = 1;
-               $compl = $ctr->{$cc}->{name};
+               $compl = $locales->{country}->{$cc}->{name};
            } else {
-               foreach my $cc (keys %$ctr) {
-                   my $ct = $ctr->{$cc}->{name};
-                   if ($ct =~ m/^\Q$text\E.*$/i) {
+               for my $country (values $locales->{country}->%*) {
+                   if ($country->{name} =~ m/^\Q$text\E.*$/i) {
                        $found++;
-                       $compl = $ct;
+                       $compl = $country->{name};
                    }
                    last if $found > 1;
                }
@@ -2237,19 +2228,20 @@ sub create_country_view {
        return undef;
     });
 
-    my $ls = Gtk3::ListStore->new('Glib::String');
-    foreach my $cc (sort {$ctr->{$a}->{name} cmp $ctr->{$b}->{name} } keys %$ctr) {
-       my $iter = $ls->append();
-       $ls->set ($iter, 0, $ctr->{$cc}->{name});
+    my $country_store = Gtk3::ListStore->new('Glib::String');
+    my $countries = $locales->{country};
+    for my $cc (sort { $countries->{$a}->{name} cmp $countries->{$b}->{name} } keys %$countries) {
+       my $iter = $country_store->append();
+       $country_store->set($iter, 0, $countries->{$cc}->{name});
     }
-    $c->set_model ($ls);
+    $c->set_model($country_store);
 
     $w->set_completion ($c);
 
-    my $hbox =  Gtk3::HBox->new(0, 0);
+    my $hbox =  Gtk3::Box->new('horizontal', 0);
 
     $label = Gtk3::Label->new("Country");
-    $label->set_alignment(1, 0.5);
+    $label->set_xalign(1.0);
     $label->set_size_request(150, -1);
     $hbox->pack_start($label, 0, 0, 10);
     $hbox->pack_start($w, 0, 0, 0);
@@ -2258,8 +2250,8 @@ sub create_country_view {
     $vbox->pack_start($hbox2, 0, 0, 5);
     $vbox->pack_start($hbox3, 0, 0, 5);
 
-    if ($country && $ctr->{$country}) {
-       $w->set_text ($ctr->{$country}->{name});
+    if ($country && (my $entry = $locales->{country}->{$country})) {
+       $w->set_text($entry->{name});
     }
 
     $inbox->show_all;
@@ -2269,7 +2261,7 @@ sub create_country_view {
 
        my $text = $w->get_text;
 
-       if (my $cc = $countryhash->{lc($text)}) {
+       if (my $cc = $locales->{countryhash}->{lc($text)}) {
            $country = $cc;
            $step_number++;
            create_password_view();
@@ -2313,7 +2305,7 @@ my $create_label_widget_grid = sub {
        my $widget = @$labeled_widgets[$i+1];
        my $label = Gtk3::Label->new(@$labeled_widgets[$i]);
        $label->set_visible(1);
-       $label->set_alignment (1, 0.5);
+       $label->set_xalign(1.0);
        $grid->attach($label, 0, $row, 1, 1);
        $widget->set_visible(1);
        $grid->attach($widget, 1, $row, 1, 1);
@@ -2541,7 +2533,7 @@ sub create_hdoption_view {
 
     # Filesystem type
     my $label0 = Gtk3::Label->new("Filesystem");
-    $label0->set_alignment (1, 0.5);
+    $label0->set_xalign(1.0);
     $grid->attach($label0, 0, $row, 1, 1);
 
     my $fstypecb = Gtk3::ComboBoxText->new();
@@ -2571,7 +2563,7 @@ sub create_hdoption_view {
 
     $row++;
 
-    my $sep = Gtk3::HSeparator->new();
+    my $sep = Gtk3::Separator->new('horizontal');
     $sep->set_visible(1);
     $grid->attach($sep, 0, $row, 2, 1);
     $row++;
@@ -2587,7 +2579,7 @@ sub create_hdoption_view {
     # size compute
     my $hdsize = 0;
     if ( -b $target_hd) {
-       $hdsize = int(hd_size($target_hd) / (1024 * 1024.0)); # size in GB
+       $hdsize = int(Proxmox::Sys::Block::hd_size($target_hd) / (1024 * 1024.0)); # size in GB
     } elsif ($target_hd) {
        $hdsize = int((-s $target_hd) / (1024 * 1024 * 1024.0));
     }
@@ -2692,7 +2684,7 @@ sub create_hdoption_view {
        &$switch_view();
     });
 
-    my $sep2 = Gtk3::HSeparator->new();
+    my $sep2 = Gtk3::Separator->new('horizontal');
     $sep2->set_visible(1);
     $contarea->pack_end($sep2, 1, 1, 10);
 
@@ -2874,9 +2866,9 @@ sub create_hdsel_view {
 
     cleanup_view();
 
-    my $vbox =  Gtk3::VBox->new(0, 0);
+    my $vbox =  Gtk3::Box->new('vertical', 0);
     $inbox->pack_start($vbox, 1, 0, 0);
-    my $hbox =  Gtk3::HBox->new(0, 0);
+    my $hbox =  Gtk3::Box->new('horizontal', 0);
     $vbox->pack_start($hbox, 0, 0, 10);
 
     my $cached_disks = get_cached_disks();
@@ -2961,12 +2953,12 @@ sub create_extract_view {
     $prev_btn->set_sensitive(0);
     $prev_btn->hide();
 
-    my $vbox =  Gtk3::VBox->new(0, 0);
+    my $vbox =  Gtk3::Box->new('vertical', 0);
     $inbox->pack_start ($vbox, 1, 0, 0);
-    my $hbox =  Gtk3::HBox->new(0, 0);
+    my $hbox =  Gtk3::Box->new('horizontal', 0);
     $vbox->pack_start ($hbox, 0, 0, 10);
 
-    my $vbox2 =  Gtk3::VBox->new(0, 0);
+    my $vbox2 =  Gtk3::Box->new('vertical', 0);
     $hbox->pack_start ($vbox2, 0, 0, 0);
 
     $progress_status = Gtk3::Label->new('');
@@ -2989,7 +2981,7 @@ sub create_extract_view {
 
     $next->set_sensitive(1);
 
-    set_next("_Reboot", sub { exit (0); } );
+    set_next("_Reboot", sub { app_quit(0); } );
 
     if ($err) {
        display_html("fail.htm");
@@ -3004,7 +2996,7 @@ sub create_extract_view {
                    $autoreboot_seconds--;
                    display_html("success.htm");
                } else {
-                   exit(0);
+                   app_quit(0);
                }
            });
        }
@@ -3042,7 +3034,7 @@ $ipconf = Proxmox::Sys::Net::get_ip_config();
 
 $country = detect_country() if $ipconf->{default} || is_test_mode();
 
-if (!defined($cmap->{country}->{$country})) {
+if (!defined($env->{locales}->{country}->{$country})) {
     log_warn("ignoring detected country '$country', invalid or unknown\n");
     $country = undef;
 }
@@ -3059,7 +3051,7 @@ my $initial_error = 0;
        print "no harddisks found\n";
        $initial_error = 1;
        display_html("nohds.htm");
-       set_next("Reboot", sub { exit(0); } );
+       set_next("Reboot", sub { app_quit(0); } );
     } else {
        foreach my $hd (@$cached_disks) {
            my ($disk, $devname) = @$hd;
@@ -3073,16 +3065,11 @@ if (!$initial_error && (scalar keys %{ $ipconf->{ifaces} } == 0)) {
     print "no network interfaces found\n";
     $initial_error = 1;
     display_html("nonics.htm");
-    set_next("Reboot", sub { exit(0); } );
+    set_next("Reboot", sub { app_quit(0); } );
 }
 
 create_intro_view () if !$initial_error;
 
 Gtk3->main;
 
-# reap left over zombie processes
-while ((my $child = waitpid(-1, POSIX::WNOHANG)) > 0) {
-    print "reaped child $child\n";
-}
-
-exit 0;
+app_quit(0);