From f9eacfa91548bbca696c78883cb939f767a7b43a Mon Sep 17 00:00:00 2001 From: Pascal Terjan Date: Tue, 3 Apr 2012 23:29:50 +0000 Subject: Drop check_build_chroot, its only goal is to force rebuilding tarball even if no package has changed over 7 days Also, do not recreate chroot from Urpmi.pm --- iurt2 | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) (limited to 'iurt2') diff --git a/iurt2 b/iurt2 index 159a32f..892ef47 100755 --- a/iurt2 +++ b/iurt2 @@ -39,7 +39,7 @@ use URPM; use Iurt::DKMS; use Iurt::Urpmi; -use Iurt::Chroot qw(add_local_user create_temp_chroot remove_chroot clean_all_chroot_tmp check_build_chroot clean_chroot); +use Iurt::Chroot qw(add_local_user create_temp_chroot remove_chroot clean_all_chroot_tmp create_build_chroot clean_chroot); use Iurt::Process qw(perform_command kill_for_good sudo); use Iurt::Mail qw(sendmail); use Iurt::Util qw(plog_init plog); @@ -716,7 +716,7 @@ $run{chroot_tar} = $chroot_tar; # even in use_old_chroot mode we create the chroot if it does not exist (useful # if the option is used for the first time if ($run{chroot} || !-d "$chroot/dev") { - check_build_chroot($chroot, $chroot_tar, \%run, $config) or die "FATAL $program_name: could not prepare initial chroot"; + create_build_chroot($chroot, $chroot_tar, \%run, $config) or die "FATAL $program_name: could not prepare initial chroot"; } # now exit if there is nothing to do and it was just a cleaning pass @@ -864,9 +864,14 @@ retry: ($ret, $srpm, $spec) = $urpmi->recreate_srpm(\%run, $config, $chroot_tmp, $dir, $srpm, $luser, $retry); if ($ret == -1) { - $retry = 1; - goto retry; - } elsif (!$ret) { + if (create_build_chroot($run->{chroot_path}, $run->{chroot_tar}, $run, $config)) { + $retry = 1; + goto retry; + } else { + $ret = 0; + } + } + if (!$ret) { # CM: experimental: fail if we can't regenerate the srpm # This should eliminate bouncers that block the input queue # -- cgit v1.2.1