Update Debian and Ubuntu Apache configuration.

Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694
Committed: https://crrev.com/a13cddf67f1d46f26103cc2f6704086d9d9a812e
Cr-Commit-Position: refs/heads/master@{#432542}

[Jack Bates, 2016-11-15 02:25:46]

Description was changed from

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Utopic and earlier.

BUG=621694
==========

to

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Utopic and earlier.

BUG=621694
==========

[Jack Bates, 2016-11-15 02:25:47]

jack@nottheoilrig.com changed reviewers:
+ dpranke@chromium.org

[Jack Bates, 2016-11-15 09:49:45]

Description was changed from

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Utopic and earlier.

BUG=621694
==========

to

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694
==========

[Jack Bates, 2016-11-15 09:50:16]

Patchset #1 (id:1) has been deleted

[Jack Bates, 2016-11-15 15:48:47]

The latest CL drops the duplicated configuration.

https://codereview.chromium.org/2502893002/diff/40001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/apache_config/debian-httpd-2.4.conf
(left):

https://codereview.chromium.org/2502893002/diff/40001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/apache_config/debian-httpd-2.4.conf:116:
<IfModule mod_php5.c>
The module is loaded unconditionally, so <IfModule> isn't necessary.
(There'll be a fatal error up above before this has any effect.)

[qyearsley, 2016-11-15 17:12:37]

On 2016/11/15 at 15:48:47, jack wrote:
> The latest CL drops the duplicated configuration.
> 
>
https://codereview.chromium.org/2502893002/diff/40001/third_party/WebKit/Tool...
> File third_party/WebKit/Tools/Scripts/apache_config/debian-httpd-2.4.conf
(left):
> 
>
https://codereview.chromium.org/2502893002/diff/40001/third_party/WebKit/Tool...
> third_party/WebKit/Tools/Scripts/apache_config/debian-httpd-2.4.conf:116:
<IfModule mod_php5.c>
> The module is loaded unconditionally, so <IfModule> isn't necessary.
> (There'll be a fatal error up above before this has any effect.)

Looks like a good simple solution to me, LGTM

[Dirk Pranke, 2016-11-15 23:19:29]

lgtm also.

[Jack Bates, 2016-11-16 00:18:36]

The CQ bit was checked by jack@nottheoilrig.com

[commit-bot: I haz the power, 2016-11-16 00:19:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 01:14:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-16 01:14:14]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Jack Bates, 2016-11-16 16:30:39]

The CQ bit was checked by jack@nottheoilrig.com

[commit-bot: I haz the power, 2016-11-16 16:31:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 17:41:54]

Description was changed from

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694
==========

to

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694
==========

[commit-bot: I haz the power, 2016-11-16 17:41:55]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-11-16 17:49:40]

Description was changed from

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694
==========

to

==========
Update Debian and Ubuntu Apache configuration.

Support PHP 7 for Stretch, Xenial and later,
and version 5 for Jessie, Wily and earlier.

BUG=621694

Committed: https://crrev.com/a13cddf67f1d46f26103cc2f6704086d9d9a812e
Cr-Commit-Position: refs/heads/master@{#432542}
==========

[commit-bot: I haz the power, 2016-11-16 17:49:41]

Patchset 2 (id:??) landed as
https://crrev.com/a13cddf67f1d46f26103cc2f6704086d9d9a812e
Cr-Commit-Position: refs/heads/master@{#432542}