Comment 0 for bug 1782806

Revision history for this message
Tim Bishop (tdb) wrote :

The following line appears to have a typo:

a2query -m "$mpm_$MPM" > /dev/null 2>&1 || a2query_ret=$?

It should read:

a2query -m "mpm_$MPM" > /dev/null 2>&1 || a2query_ret=$?

Since $mpm is not defined. Later on there are references to enabling and disabling "mpm_$MPM".

https://salsa.debian.org/apache-team/apache2/blob/master/debian/debhelper/apache2-maintscript-helper#L290

This appears to trip up the Puppet apache module since it creates a prefork module (rather than mpm_prefork), which results in the above query returning a positive response. This is what's happening in bug #1771934.

Fix is obvious and trivial so can hopefully be implemented soon. Appears only to affect bionic since xenial had different code.