The following commit has been merged in the master branch:
commit af77eb72143642921585ed38efb5ab3b8158ab7f
Author: Guillem Jover <[EMAIL PROTECTED]>
Date: Sat Dec 6 23:03:54 2008 +0200
scripts: Do not use sprintf for functions supporting variable args
diff --git a/ChangeLog b/ChangeLog
index 77e21a8..8b9d1fd 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
2008-12-08 Guillem Jover <[EMAIL PROTECTED]>
+ * scripts/Dpkg/Deps.pm (parse): Do not use sprintf for warning.
+ (implies): Do not use sprintf for internerr.
+
+2008-12-08 Guillem Jover <[EMAIL PROTECTED]>
+
* dpkg-deb/build.c (do_build): Use internerr instead of ohshit. Do not
mark the string for translation.
* src/trigcmd.c (do_check): Use internerr instead of abort.
diff --git a/scripts/Dpkg/Deps.pm b/scripts/Dpkg/Deps.pm
index f9e4193..e596880 100644
--- a/scripts/Dpkg/Deps.pm
+++ b/scripts/Dpkg/Deps.pm
@@ -310,7 +310,7 @@ sub parse {
foreach my $dep_or (split(/\s*\|\s*/m, $dep_and)) {
my $dep_simple = Dpkg::Deps::Simple->new($dep_or);
if (not defined $dep_simple->{package}) {
- warning(sprintf(_g("can't parse dependency %s"), $dep_and));
+ warning(_g("can't parse dependency %s"), $dep_and);
return undef;
}
$dep_simple->{arches} = undef if not $options{use_arch};
@@ -623,7 +623,8 @@ sub implies {
}
return $res;
} else {
- internerr(sprintf(_g("Dpkg::Deps::Simple can't evaluate implication
with a %s!"), ref($o)));
+ internerr(_g("Dpkg::Deps::Simple can't evaluate implication with a
%s!"),
+ ref($o));
}
}
--
dpkg's main repository
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
