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  (community) Needs Information
Review via email:

Description of the change


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 file to get installed.

I plan to include this SRU with the xenial fix for lp #1832182 (see, 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 You'd really need to go out of your way to make Apache read 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


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
1diff --git a/debian/apache2.install b/debian/apache2.install
2index 9ef8ca9..8941658 100644
3--- a/debian/apache2.install
4+++ b/debian/apache2.install
5@@ -1,5 +1,8 @@
6 debian/bash_completion/a2enmod /usr/share/bash-completion/completions/
7-debian/config-dir/* /etc/apache2
8+debian/config-dir/*-available /etc/apache2
9+debian/config-dir/*.conf /etc/apache2
10+debian/config-dir/envvars /etc/apache2
11+debian/config-dir/magic /etc/apache2
12 debian/a2enmod /usr/sbin
13 debian/apache2ctl /usr/sbin
14 debian/a2query /usr/sbin
15diff --git a/debian/changelog b/debian/changelog
16index 659f7b8..6a9da9d 100644
17--- a/debian/changelog
18+++ b/debian/changelog
19@@ -1,3 +1,11 @@
20+apache2 (2.4.18-2ubuntu3.18) xenial; urgency=medium
22+ * d/apache2.install: List confdir contents explicitly. Avoids
23+ installing *.in templates.
24+ (LP: #1899611)
26+ -- Bryce Harrington <> Mon, 16 Nov 2020 16:23:23 -0800
28 apache2 (2.4.18-2ubuntu3.17) xenial-security; urgency=medium
30 * SECURITY UPDATE: mod_rewrite redirect issue


People subscribed via source and target branches