Attachment 35674 Details for Bug 13486 – Bug 13486: Apache version empty in about page if Apache 2.4

[patch] Bug 13486: Apache version empty in about page if Apache 2.4

Bug-13486-Apache-version-empty-in-about-page-if-Ap.patch (text/plain), 1.64 KB, created by Tomás Cohen Arazi (tcohen) on 2015-02-05 17:38:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2015-02-05 17:38:04 UTC

Size: 1.64 KB

patch

obsolete

>From 7efb4d0a46a2bdb945affe49c230b1f4a262a557 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@gmail.com>
>Date: Thu, 5 Feb 2015 14:31:55 -0300
>Subject: [PATCH] Bug 13486: Apache version empty in about page if Apache 2.4
>
>This patch makes about.pl query the running Apache version using
>the apache2ctl command. I tested it on Apache 2.2 and is backwards
>compatible (it is present in both).
>
>To test:
>( On an Apache 2.4 setup )
>- Go to the about page
>=> FAIL: "Apache version:" is empty
>- Apply the patch
>- Reload the page
>=> SUCCESS: "Apache version:" shows the correct version.
>- Sign off :-D
>
>If you have an Apache 2.2 setup to try it, verify that there are no regressions
>(trivial because you can run apache2ctl manually and verify the first result is the expecte).
>
>Regards
>
>PS. I even changed the order in which Apache version is tested because
>most people is using Debian/Ubuntu and it was the last option.
>---
> about.pl | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
>diff --git a/about.pl b/about.pl
>index 9fd1545..57a6f53 100755
>--- a/about.pl
>+++ b/about.pl
>@@ -55,9 +55,10 @@ if ($^O ne 'VMS') {
> }
> my $perlVersion   = $];
> my $mysqlVersion  = `mysql -V`;
>-my $apacheVersion = `httpd -v 2> /dev/null`;
>-$apacheVersion = `httpd2 -v 2> /dev/null` unless $apacheVersion;
>-$apacheVersion = (`/usr/sbin/apache2 -V`)[0] unless $apacheVersion;
>+# Get Apache version
>+my $apacheVersion = (`/usr/sbin/apache2ctl -V`)[0];
>+$apacheVersion    = `httpd2 -v 2> /dev/null` unless $apacheVersion;
>+$apacheVersion    = `httpd -v 2> /dev/null` unless $apacheVersion;
> my $zebraVersion = `zebraidx -V`;
> 
> # Additional system information for warnings
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13486: 34621 | 35674 | 39092 | 39094 | 39385