diff options
| -rw-r--r-- | NEWS | 2 | ||||
| -rw-r--r-- | lib/Iurt/Urpmi.pm | 5 |
2 files changed, 4 insertions, 3 deletions
@@ -1,3 +1,5 @@ +- iurt: Fix getting srpm name when parsing the spec causes some output on stdout + 0.9.3 - ulri: Fix a crash on build failure. This caused to not send failure emails and not cleanup packages for other architectures. diff --git a/lib/Iurt/Urpmi.pm b/lib/Iurt/Urpmi.pm index aa14c57..a5162ce 100644 --- a/lib/Iurt/Urpmi.pm +++ b/lib/Iurt/Urpmi.pm @@ -449,7 +449,7 @@ sub get_srpm_name { $log_spool_existed = 0; mkdir $log_spool; } - if (!perform_command(qq(chroot $chroot_tmp su $luser -c "rpmspec -q --qf %{NVR}.src.rpm --srpm /home/$luser/rpmbuild/SPECS/$spec > /home/$luser/rpmbuild/SPECS/$spec.srpm_name"), + if (!perform_command(qq(chroot $chroot_tmp su $luser -c "rpmspec -q --qf %{NVR}.src.rpm --srpm /home/$luser/rpmbuild/SPECS/$spec | tail -n1 > /home/$luser/rpmbuild/SPECS/$spec.srpm_name"), $run, $config, use_iurt_root_command => 1, log => $log_spool, @@ -596,7 +596,6 @@ sub install_dynamic_buildrequires { return; } my $nosrc = $new_srpm =~ s/src.rpm$/buildreqs.nosrc.rpm/r; - while (1) { # There is no way with perform_command to get the actual error code. # We can however tell it that error 11 (missing BuildRequires) is ok so @@ -631,7 +630,7 @@ sub install_dynamic_buildrequires { unlink("$chroot_tmp/home/$luser/rpmbuild/SRPMS/$nosrc"); } else { # This was a failure for another reason, no point retrying - plog('ERROR', "Failed to generate dynamic BuildRequires."); + plog('ERROR', "Failed to generate dynamic BuildRequires ($chroot_tmp/home/$luser/rpmbuild/SRPMS/$nosrc does not exist)."); $run->{status}{$srpm} = 'build_failure'; return 0; } |