Package: dpkg Severity: minor Tags: patch User: ubuntu-de...@lists.ubuntu.com Usertags: origin-ubuntu jammy ubuntu-patch
Dear Maintainer, dpkg-source will do a check for a scenario where the maintainer field in d/control does not contain "ubuntu" and the DEBEMAIL environment variable ends in @ubuntu.com. There are some Canonical employees that use an @canonical.com email address instead, so we should expand this check to also trigger on DEBEMAIL values that end in @canonical.com -- Package-specific info: System tainted due to merged-usr-via-aliased-dirs. *** /tmp/tmpb4294xf9/bug_body In Ubuntu, the attached patch was applied to achieve the following: Have dpkg-source fail in the aforementioned scenario, rather than a warning. * scripts/Dpkg/Vendor/Ubuntu.pm: when checking for the correct maintainer field, also look for @canonical.com email addresses (LP: #1951988) Thanks for considering the patch. -- System Information: Debian Release: bookworm/sid APT prefers jammy APT policy: (500, 'jammy') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 5.13.0-20-generic (SMP w/32 CPU threads) Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_WARN, TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set Shell: /bin/sh linked to /usr/bin/dash Init: systemd (via /run/systemd/system) LSM: AppArmor: enabled
diff -Nru dpkg-1.20.9ubuntu2/debian/control dpkg-1.20.9ubuntu3/debian/control --- dpkg-1.20.9ubuntu2/debian/control 2021-04-14 05:32:22.000000000 -0500 +++ dpkg-1.20.9ubuntu3/debian/control 2021-11-23 09:42:41.000000000 -0600 @@ -1,8 +1,7 @@ Source: dpkg Section: admin Priority: required -Maintainer: Ubuntu Developers <ubuntu-devel-disc...@lists.ubuntu.com> -XSBC-Original-Maintainer: Dpkg Developers <debian-d...@lists.debian.org> +Maintainer: Dpkg Developers <debian-d...@lists.debian.org> Uploaders: Guillem Jover <guil...@debian.org> Homepage: https://wiki.debian.org/Teams/Dpkg Vcs-Browser: https://git.dpkg.org/cgit/dpkg/dpkg.git diff -Nru dpkg-1.20.9ubuntu2/scripts/Dpkg/Vendor/Ubuntu.pm dpkg-1.20.9ubuntu3/scripts/Dpkg/Vendor/Ubuntu.pm --- dpkg-1.20.9ubuntu2/scripts/Dpkg/Vendor/Ubuntu.pm 2021-04-14 05:32:22.000000000 -0500 +++ dpkg-1.20.9ubuntu3/scripts/Dpkg/Vendor/Ubuntu.pm 2021-11-23 09:42:41.000000000 -0600 @@ -55,7 +55,7 @@ if (defined($fields->{'Version'}) and defined($fields->{'Maintainer'}) and $fields->{'Version'} =~ /ubuntu/) { if ($fields->{'Maintainer'} !~ /ubuntu/i) { - if (length $ENV{DEBEMAIL} and $ENV{DEBEMAIL} =~ /\@ubuntu\.com/) { + if (length $ENV{DEBEMAIL} and $ENV{DEBEMAIL} =~ /\@ubuntu\.com|\@canonical\.com/) { error(g_('Version number suggests Ubuntu changes, but Maintainer: does not have Ubuntu address')); } else { warning(g_('Version number suggests Ubuntu changes, but Maintainer: does not have Ubuntu address')); File /tmp/6yZkN9jpdE/dpkg-1.20.9ubuntu2/tests/t-unpack-fifo/pkg-fifo/test-fifo is a fifo while file /tmp/u2tkW_fatS/dpkg-1.20.9ubuntu3/tests/t-unpack-fifo/pkg-fifo/test-fifo is a fifo File /tmp/6yZkN9jpdE/dpkg-1.20.9ubuntu2/tests/t-unpack-hardlink/pkg-hardlink/test-fifo-link0 is a fifo while file /tmp/u2tkW_fatS/dpkg-1.20.9ubuntu3/tests/t-unpack-hardlink/pkg-hardlink/test-fifo-link0 is a fifo File /tmp/6yZkN9jpdE/dpkg-1.20.9ubuntu2/tests/t-unpack-hardlink/pkg-hardlink/test-fifo-link1 is a fifo while file /tmp/u2tkW_fatS/dpkg-1.20.9ubuntu3/tests/t-unpack-hardlink/pkg-hardlink/test-fifo-link1 is a fifo