Package: release.debian.org
Severity: normal
Tags: jessie
User: release.debian....@packages.debian.org
Usertags: pu

Hi,

please review apache2/2.4.10-10+deb8u4 for inclusion in s-p-u. It
includes a fixed version of the deferred mpm switch patch that was
pulled from the last s-p-u upload due to a bug. It also fixes some other
annoying but easy to fix issues.

Changelog:

 * Add versioned replaces/breaks for libapache2-mod-macro to apache2,
   for the config files in /etc. Closes: #806326
 * Fix split-logfile to work with current perl. Closes: #803472
 * Fix tests on deferred mpm switch. Add special casing for mpm_itk,
   which is not an mpm anymore, despite the name. Closes: #789914
   Closes: #791902
 * Fix secondary-init-script to not source the main init script with
   'set -e'. Closes: #803177

Debdiff is attached.

Cheers,
Stefan
diff -Nru apache2-2.4.10/debian/a2enmod apache2-2.4.10/debian/a2enmod
--- apache2-2.4.10/debian/a2enmod	2015-08-27 23:03:43.000000000 +0200
+++ apache2-2.4.10/debian/a2enmod	2015-11-28 14:08:16.000000000 +0100
@@ -430,7 +430,7 @@
     if ( $? != 0 ) {
 
         # config doesn't work
-        if ( -e "$enabldir/mpm_prefork.load" || -e "$enabldir/mpm_itk.load" )
+        if ( -e "$enabldir/mpm_prefork.load" )
         {
             return 0;
         }
diff -Nru apache2-2.4.10/debian/a2query.in apache2-2.4.10/debian/a2query.in
--- apache2-2.4.10/debian/a2query.in	2015-08-07 23:33:37.000000000 +0200
+++ apache2-2.4.10/debian/a2query.in	2015-11-28 14:07:43.000000000 +0100
@@ -209,10 +209,10 @@
 		my $file = $_;
 		next if $file !~ m/\.load$/;
 		$file =~ s/\.load//;
-		if ($file =~ /mpm_(\w+)/ && $file ne 'mpm_itk')
+		if ($file =~ /mpm_(?:event|worker|prefork)/)
 		{
 			$MPM = $1 if $MPM eq 'invalid';
-			if(grep { $_ =~ m/^mpm_/ } @MODULES)
+			if(grep { $_ =~ m/^mpm_(?:event|worker|prefork)/ } @MODULES)
 			{
 				fail("There is more than one MPM loaded. Do not proceed due to undefined results", 1);
 			}
diff -Nru apache2-2.4.10/debian/apache2.postinst apache2-2.4.10/debian/apache2.postinst
--- apache2-2.4.10/debian/apache2.postinst	2015-08-28 18:11:59.000000000 +0200
+++ apache2-2.4.10/debian/apache2.postinst	2015-11-28 14:08:41.000000000 +0100
@@ -432,21 +432,21 @@
 			esac
 			;;
 		apache2_switch_mpm)
-			local MPM="mpm_$ARG1"
-			local CUR_MPM="$(ls /etc/apache2/mods-enabled/mpm_*.load)"
+			local MPM="$ARG1"
+			local CUR_MPM="$(ls /etc/apache2/mods-enabled/mpm_*.load | grep -e event -e prefork -e worker)"
 			CUR_MPM="${CUR_MPM##*/mpm_}"
 			CUR_MPM="${CUR_MPM%.load}"
-			if [ ! -e /etc/apache2/mods-available/$MPM.load ] ; then
-				msg "error" "$MPM not found in 'apache2_switch_mpm $ARG1' for package $PACKAGE"
+			if [ ! -e /etc/apache2/mods-available/mpm_$MPM.load ] ; then
+				msg "error" "mpm $MPM not found in 'apache2_switch_mpm $ARG1' for package $PACKAGE"
 				error=true
-			elif [ ! -e /etc/apache2/mods-enabled/$MPM.load ] ; then
-				msg "info" "$MPM: No action required"
+			elif [ -e /etc/apache2/mods-enabled/mpm_$MPM.load ] ; then
+				msg "info" "Switch to mpm $MPM for package $PACKAGE: No action required"
 			else
-				msg "info" "Switch to $MPM for package $PACKAGE"
+				msg "info" "Switch to mpm $MPM for package $PACKAGE"
 				if ! a2dismod -m -q "mpm_$CUR_MPM" ||
 				   ! a2enmod -m -q "mpm_$MPM"
 				then
-					msg "error" "Switching to $MPM failed"
+					msg "error" "Switching to mpm $MPM failed"
 					error=true
 				fi
 			fi
diff -Nru apache2-2.4.10/debian/changelog apache2-2.4.10/debian/changelog
--- apache2-2.4.10/debian/changelog	2015-08-28 18:26:05.000000000 +0200
+++ apache2-2.4.10/debian/changelog	2015-11-28 15:02:23.000000000 +0100
@@ -1,3 +1,16 @@
+apache2 (2.4.10-10+deb8u4) jessie; urgency=medium
+
+  * Add versioned replaces/breaks for libapache2-mod-macro to apache2,
+    for the config files in /etc. Closes: #806326
+  * Fix split-logfile to work with current perl. Closes: #803472
+  * Fix tests on deferred mpm switch. Add special casing for mpm_itk,
+    which is not an mpm anymore, despite the name. Closes: #789914
+    Closes: #791902
+  * Fix secondary-init-script to not source the main init script with 'set -e'.
+    Closes: #803177
+
+ -- Stefan Fritsch <s...@debian.org>  Sat, 28 Nov 2015 15:02:23 +0100
+
 apache2 (2.4.10-10+deb8u3) jessie; urgency=medium
 
   * Revert fix for deferred mpm switch for now, because it is at least not
diff -Nru apache2-2.4.10/debian/control apache2-2.4.10/debian/control
--- apache2-2.4.10/debian/control	2015-08-27 23:03:43.000000000 +0200
+++ apache2-2.4.10/debian/control	2015-11-28 13:59:22.000000000 +0100
@@ -20,7 +20,8 @@
  perl, mime-support, apache2-bin (= ${binary:Version}),
  apache2-utils (>= 2.4), apache2-data (= ${source:Version}),
  ${perl:Depends}
-Replaces: apache2.2-common
+Replaces: apache2.2-common, libapache2-mod-macro (<< 1:2.4.6-1~)
+Breaks: libapache2-mod-macro (<< 1:2.4.6-1~)
 Conflicts: apache2.2-common (<< 2.3~)
 Provides: httpd, httpd-cgi
 Recommends: ssl-cert
diff -Nru apache2-2.4.10/debian/patches/series apache2-2.4.10/debian/patches/series
--- apache2-2.4.10/debian/patches/series	2015-08-27 23:03:43.000000000 +0200
+++ apache2-2.4.10/debian/patches/series	2015-11-28 13:59:22.000000000 +0100
@@ -17,3 +17,4 @@
 define_restarts.diff
 mpm_event_graceful_restart_deadlock.diff
 mpm_event_crash.diff
+split_logfile-strict.patch
diff -Nru apache2-2.4.10/debian/patches/split_logfile-strict.patch apache2-2.4.10/debian/patches/split_logfile-strict.patch
--- apache2-2.4.10/debian/patches/split_logfile-strict.patch	1970-01-01 01:00:00.000000000 +0100
+++ apache2-2.4.10/debian/patches/split_logfile-strict.patch	2015-11-28 13:59:22.000000000 +0100
@@ -0,0 +1,34 @@
+# Fix for 'use strict' in current perl versions
+# https://svn.apache.org/viewvc?view=revision&revision=1651095
+--- apache2.orig/support/split-logfile.in
++++ apache2/support/split-logfile.in
+@@ -29,7 +29,7 @@
+ use strict;
+ use warnings;
+ 
+-my %is_open = ();
++my %log_file = ();
+ 
+ while (my $log_line = <STDIN>) {
+     #
+@@ -54,10 +54,9 @@ while (my $log_line = <STDIN>) {
+     # If the log file for this virtual host isn't opened
+     # yet, do it now.
+     #
+-    if (! $is_open{$vhost}) {
+-        open $vhost, ">>${vhost}.log"
++    if (! $log_file{$vhost}) {
++        open $log_file{$vhost}, ">>${vhost}.log"
+             or die ("Can't open ${vhost}.log");
+-        $is_open{$vhost} = 1;
+     }
+     #
+     # Strip off the first token (which may be null in the
+@@ -65,6 +64,6 @@ while (my $log_line = <STDIN>) {
+     # record to the current log file.
+     #
+     $log_line =~ s/^\S*\s+//;
+-    printf $vhost "%s", $log_line;
++    print {$log_file{$vhost}} $log_line;
+ }
+ exit 0;
diff -Nru apache2-2.4.10/debian/secondary-init-script apache2-2.4.10/debian/secondary-init-script
--- apache2-2.4.10/debian/secondary-init-script	2015-08-02 00:21:24.000000000 +0200
+++ apache2-2.4.10/debian/secondary-init-script	2015-11-28 14:15:20.000000000 +0100
@@ -9,8 +9,6 @@
 # Short-Description: Start/stop apache2 web server (config /etc/apache2-XXX)
 ### END INIT INFO
 
-set -e
-
 #
 # this init script can be installed as /etc/init.d/apache2-XXX
 #

Reply via email to