Merge ~bryce/ubuntu/+source/apache2:sru-lp1565744-apache-lock-dir into ubuntu/+source/apache2:ubuntu/xenial-devel

Proposed by Bryce Harrington
Status: Needs review
Proposed branch: ~bryce/ubuntu/+source/apache2:sru-lp1565744-apache-lock-dir
Merge into: ubuntu/+source/apache2:ubuntu/xenial-devel
Diff against target: 30 lines (+12/-1)
2 files modified
debian/apache2.install (+4/-1)
debian/changelog (+8/-0)
Reviewer Review Type Date Requested Status
Christian Ehrhardt  Needs Information
Review via email: mp+393891@code.launchpad.net

Description of the change

PPA: https://launchpad.net/~bryce/+archive/ubuntu/apache2-sru-lp1899611-conf-in/

Backports a Debian fix for installation of config files, so can avoid installing *.in templates. Fixes an issue from an earlier SRU that causes an apache2.conf.in file to get installed.

I plan to include this SRU with the xenial fix for lp #1832182 (see https://code.launchpad.net/~bryce/ubuntu/+source/apache2/+git/apache2/+merge/393678), but am MP'ing this separately for review convenience.

I haven't filled in the SRU text yet, but the bug report is pretty clear and the test case obvious. I do plan to fill in a proper SRU before upload.

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Straightforward case and fix, I found no issues other than "on its own" it would be debatable if it is worth an update. But since you plan to upload the fixes together that is ok.

One question thou, since we have placed things in /etc/ they will be config files.
Therefore "not delivering" them will not remove them from disk.

Is there a chance that people started to (wrongly) use that file and it would be active?

If not I think we should we also add an rm_conffile along the fix.

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

It couldn't be active because it's not a validly formatted config file so would generate an error.

Apache by default uses apache2.conf, not apache2.conf.in. You'd really need to go out of your way to make Apache read apache2.conf.in instead. So I think the utility of removing the file is going to be pretty low. And yeah, I wouldn't have bothered with this as SRU except that I have another one it can easily ride in on.

Unmerged commits

a49d30c... by Bryce Harrington

changelog

0ef095b... by Bryce Harrington

* d/apache2.install: List confdir contents explicitly. Avoids installing *.in templates. (LP: #1899611)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/apache2.install b/debian/apache2.install
index 9ef8ca9..8941658 100644
--- a/debian/apache2.install
+++ b/debian/apache2.install
@@ -1,5 +1,8 @@
1debian/bash_completion/a2enmod /usr/share/bash-completion/completions/1debian/bash_completion/a2enmod /usr/share/bash-completion/completions/
2debian/config-dir/* /etc/apache22debian/config-dir/*-available /etc/apache2
3debian/config-dir/*.conf /etc/apache2
4debian/config-dir/envvars /etc/apache2
5debian/config-dir/magic /etc/apache2
3debian/a2enmod /usr/sbin6debian/a2enmod /usr/sbin
4debian/apache2ctl /usr/sbin7debian/apache2ctl /usr/sbin
5debian/a2query /usr/sbin8debian/a2query /usr/sbin
diff --git a/debian/changelog b/debian/changelog
index 659f7b8..6a9da9d 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
1apache2 (2.4.18-2ubuntu3.18) xenial; urgency=medium
2
3 * d/apache2.install: List confdir contents explicitly. Avoids
4 installing *.in templates.
5 (LP: #1899611)
6
7 -- Bryce Harrington <bryce@canonical.com> Mon, 16 Nov 2020 16:23:23 -0800
8
1apache2 (2.4.18-2ubuntu3.17) xenial-security; urgency=medium9apache2 (2.4.18-2ubuntu3.17) xenial-security; urgency=medium
210
3 * SECURITY UPDATE: mod_rewrite redirect issue11 * SECURITY UPDATE: mod_rewrite redirect issue

Subscribers

People subscribed via source and target branches