Merge ~lucaskanashiro/ubuntu/+source/postfix:lp1872288 into ubuntu/+source/postfix:ubuntu/devel

Proposed by Lucas Kanashiro
Status: Merged
Approved by: Bryce Harrington
Approved revision: 741cc12486cafa739a4d48142283103c7c4c6627
Merged at revision: 741cc12486cafa739a4d48142283103c7c4c6627
Proposed branch: ~lucaskanashiro/ubuntu/+source/postfix:lp1872288
Merge into: ubuntu/+source/postfix:ubuntu/devel
Diff against target: 50 lines (+10/-3)
3 files modified
debian/changelog (+6/-0)
debian/configure-instance.sh (+2/-2)
debian/control (+2/-1)
Reviewer Review Type Date Requested Status
Bryce Harrington (community) Approve
Canonical Server Core Reviewers Pending
Review via email: mp+382443@code.launchpad.net

Description of the change

There is a typo in the debian/configure-instance.sh script which causes an issue when copying certificates. It was reported by an user, and the Debian maintainer pointed to the right fix here:

https://bugs.launchpad.net/ubuntu/+source/postfix/+bug/1872288

I simply cherry-picked this patch mentioned in the bug report:

https://salsa.debian.org/postfix-team/postfix-dev/-/commit/b8e0b846e34eeaaa2315ead2304824b21b01fe7a

Here is a PPA with the proposed package:

https://launchpad.net/~lucaskanashiro/+archive/ubuntu/focal-postfix-lp1872288/+packages

And autopkgtest is still happy with those changes:

autopkgtest [15:06:27]: @@@@@@@@@@@@@@@@@@@@ summary
postfix PASS

To post a comment you must log in.
Revision history for this message
Scott Kitterman (kitterman) wrote :

FWIW, looks right to me.

Revision history for this message
Bryce Harrington (bryce) wrote :

LGTM as well, +1.

The typos are obviously incorrect, from a cursory code review. `shellcheck configure-instance.sh` also notices the errors. shellcheck shows several lint warnings but no other actual errors.

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Upload sponsored:

stirling:~/ubuntu/Postfix/review-mp382443/postfix-gu$ gpg --verify ${changes_file}
gpg: Signature made Thu 16 Apr 2020 12:00:34 PM PDT
gpg: using RSA key A661100B3DAC1D4F2CAD8A54E603B2578FB8F0FB
gpg: Good signature from "Bryce Harrington <email address hidden>" [ultimate]
gpg: aka "Bryce Harrington <email address hidden>" [ultimate]
gpg: aka "Bryce Harrington <email address hidden>" [ultimate]
stirling:~/ubuntu/Postfix/review-mp382443/postfix-gu$ git ubuntu tag --upload
stirling:~/ubuntu/Postfix/review-mp382443/postfix-gu$ git push pkg upload/3.4.10-1ubuntu1
Enumerating objects: 18, done.
Counting objects: 100% (18/18), done.
Delta compression using up to 12 threads
Compressing objects: 100% (13/13), done.
Writing objects: 100% (13/13), 1.63 KiB | 277.00 KiB/s, done.
Total 13 (delta 9), reused 0 (delta 0)
To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/postfix
 * [new tag] upload/3.4.10-1ubuntu1 -> upload/3.4.10-1ubuntu1
stirling:~/ubuntu/Postfix/review-mp382443/postfix-gu$ dput ubuntu ${changes_file}
Checking signature on .changes
gpg: ../postfix_3.4.10-1ubuntu1_source.changes: Valid signature from E603B2578FB8F0FB
Checking signature on .dsc
gpg: ../postfix_3.4.10-1ubuntu1.dsc: Valid signature from E603B2578FB8F0FB
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading postfix_3.4.10-1ubuntu1.dsc: done.
  Uploading postfix_3.4.10-1ubuntu1.debian.tar.xz: done.
  Uploading postfix_3.4.10-1ubuntu1_source.buildinfo: done.
  Uploading postfix_3.4.10-1ubuntu1_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 2685f3d..86e71a2 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+postfix (3.4.10-1ubuntu1) focal; urgency=medium
7+
8+ * d/configure-instance.sh: fix typo in tls_CApath copying (LP: #1872288)
9+
10+ -- Lucas Kanashiro <lucas.kanashiro@canonical.com> Thu, 16 Apr 2020 14:48:33 -0300
11+
12 postfix (3.4.10-1) unstable; urgency=medium
13
14 [Scott Kitterman]
15diff --git a/debian/configure-instance.sh b/debian/configure-instance.sh
16index a327c84..feb20d6 100644
17--- a/debian/configure-instance.sh
18+++ b/debian/configure-instance.sh
19@@ -45,7 +45,7 @@ if [ -n "$NEED_CHROOT" ] && [ -n "$SYNC_CHROOT" ]; then
20 cd "$queue_dir"
21
22 # copy the smtp CA path if specified
23- ca_path=$($POSTCONF -hx smtp_tls_CApath)
24+ sca_path=$($POSTCONF -hx smtp_tls_CApath)
25 case "$sca_path" in
26 '') :;; # no sca_path
27 $queue_dir/*) :;; # skip stuff already in chroot
28@@ -77,7 +77,7 @@ if [ -n "$NEED_CHROOT" ] && [ -n "$SYNC_CHROOT" ]; then
29 esac
30
31 # copy the smtpd CA path if specified
32- ca_path=$($POSTCONF -hx smtpd_tls_CApath)
33+ dca_path=$($POSTCONF -hx smtpd_tls_CApath)
34 case "$dca_path" in
35 '') :;; # no dca_path
36 $queue_dir/*) :;; # skip stuff already in chroot
37diff --git a/debian/control b/debian/control
38index 5e9a483..3288242 100644
39--- a/debian/control
40+++ b/debian/control
41@@ -1,7 +1,8 @@
42 Source: postfix
43 Section: mail
44 Priority: optional
45-Maintainer: LaMont Jones <lamont@debian.org>
46+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
47+XSBC-Original-Maintainer: LaMont Jones <lamont@debian.org>
48 Uploaders: Scott Kitterman <scott@kitterman.com>
49 Standards-Version: 4.5.0
50 Homepage: http://www.postfix.org

Subscribers

People subscribed via source and target branches