summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorunknown <ex.thierry.vignaud@NANTEXPUC210048.dom-grand-ouest.justice.fr>2015-02-23 10:54:37 +0100
committerThierry Vignaud <thierry.vignaud@gmail.com>2015-02-26 19:02:31 +0100
commit3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124 (patch)
tree26a79c1ce74014b59f26600865fa72c2cfef1eb4
parentb411d0ff7aefe578dc1a9a21ba4d1c9db3d50327 (diff)
downloadurpmi-3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124.tar
urpmi-3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124.tar.gz
urpmi-3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124.tar.bz2
urpmi-3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124.tar.xz
urpmi-3d9a2bcd451b1e0f0ee9b6afe93d01c16c37a124.zip
always display the right package name on errors
fixes mga#15032 and also simplify... needs URPM >= 5.04
-rw-r--r--NEWS1
-rw-r--r--urpm/install.pm15
2 files changed, 3 insertions, 13 deletions
diff --git a/NEWS b/NEWS
index a3443f7f..708ac91c 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,4 @@
+- always display the right package name on errors (mga#15032)
- don't give a wrong package name when it is unknown in a callback (mga#15032)
Version 8.03 - 1 January 2015
diff --git a/urpm/install.pm b/urpm/install.pm
index d97c4c14..0aa29528 100644
--- a/urpm/install.pm
+++ b/urpm/install.pm
@@ -300,19 +300,8 @@ sub _get_callbacks {
$options->{callback_uninst} ||= $options->{verbose} >= 0 ? \&install_logger : $erase_logger;
$options->{callback_error} ||= sub {
- my ($urpm, undef, $id, $subtype) = @_;
- my $n;
- if (defined($id)) {
- $n = $urpm->{depslist}[$id]->fullname;
- } else {
- # We don't know which package :(
- if ($trans->NElements() == 1) {
- $n = $trans->Element_fullname(0);
- } else {
- $n = "(unknown)";
- }
- }
- $urpm->{error}("ERROR: '$subtype' failed for $n");
+ my ($urpm, undef, $id, $subtype, undef, undef, $fullname) = @_;
+ $urpm->{error}("ERROR: '$subtype' failed for $fullname");
};
if ($options->{verbose} >= 0 && $have_pkgs) {