Skip to content

Commit b538311

Browse files
author
epriestley
committedJul 5, 2016
Raise explicit setup issues about PHP 7
Summary: Ref T9640. Fixes T9888. Decline to support PHP 7 until the async signal handling issue in T11270 is resolved. Test Plan: Faked local version, got helpful error message. Reviewers: chad Reviewed By: chad Maniphest Tasks: T9640, T9888 Differential Revision: https://secure.phabricator.com/D16231
1 parent c7e7f11 commit b538311

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed
 

‎src/applications/config/check/PhabricatorPHPConfigSetupCheck.php

+17
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,23 @@ public function getExecutionOrder() {
1111
}
1212

1313
protected function executeChecks() {
14+
if (version_compare(phpversion(), 7, '>=')) {
15+
$message = pht(
16+
'This version of Phabricator does not support PHP 7. You '.
17+
'are running PHP %s.',
18+
phpversion());
19+
20+
$this->newIssue('php.version7')
21+
->setIsFatal(true)
22+
->setName(pht('PHP 7 Not Supported'))
23+
->setMessage($message)
24+
->addLink(
25+
'https://phurl.io/u/php7',
26+
pht('Phabricator PHP 7 Compatibility Information'));
27+
28+
return;
29+
}
30+
1431
$safe_mode = ini_get('safe_mode');
1532
if ($safe_mode) {
1633
$message = pht(

‎src/docs/user/installation_guide.diviner

+2-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ Beyond an operating system, you will need **a webserver**.
7070
You will also need:
7171

7272
- **MySQL**: You need MySQL. We strongly recommend MySQL 5.5 or newer.
73-
- **PHP**: You need PHP 5.2 or newer.
73+
- **PHP**: You need PHP 5.2 or newer, but note that PHP 7 is
74+
**not supported**.
7475

7576
You'll probably also need a **domain name**. In particular, you should read this
7677
note:

0 commit comments

Comments
 (0)
Failed to load comments.