The following commit has been merged in the master branch:
commit a616f22100ddc3be5835fb1d909857f39c3b231d
Author: Guillem Jover <[email protected]>
Date: Tue Apr 23 21:29:24 2013 +0200
Dpkg::Vendor::Debian: Do not use Dpkg::Vendor::Ubuntu, just require it
The fact that Dpkg::Vendor::Debian needs to use functions from
Dpkg::Vendor::Ubuntu has always been all kinds of a broken design, as
an obvious layer violation, but at least for now we avoid redefinition
of functions.
diff --git a/scripts/Dpkg/Vendor/Debian.pm b/scripts/Dpkg/Vendor/Debian.pm
index 81b5714..fcfce17 100644
--- a/scripts/Dpkg/Vendor/Debian.pm
+++ b/scripts/Dpkg/Vendor/Debian.pm
@@ -29,7 +29,6 @@ use base qw(Dpkg::Vendor::Default);
use Dpkg::Gettext;
use Dpkg::ErrorHandling;
use Dpkg::Control::Types;
-use Dpkg::Vendor::Ubuntu;
use Dpkg::BuildOptions;
use Dpkg::Arch qw(get_host_arch debarch_to_debtriplet);
@@ -61,6 +60,8 @@ sub run_hook {
}
}
+ # XXX: Layer violation...
+ require Dpkg::Vendor::Ubuntu;
my $b =
Dpkg::Vendor::Ubuntu::find_launchpad_closes($ch_info->{'Changes'});
foreach my $bug (@$b) {
$$textref .= "Bug-Ubuntu: https://bugs.launchpad.net/bugs/$bug\n";;
--
dpkg's main repository
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
