revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-23 05:50:55 +01:00
Code Issues Releases Wiki Activity

Changing the method of validating sendmail exists on the system.

Browse source 
Summary:
Removes calling `which` and instead checks the file path that PHPMailerLite uses directly (/usr/bin/sendmail).

This fixes $PATH export issues which occur on certain platforms causing un-expected errors during setup.

Test Plan:
* Run setup on server without sendmail install & error should be presented.
* Install sendmail then re-run setup & no error should be presented.

Reviewers: epriestley

Differential Revision: 375
This commit is contained in:
Damian Zaremba 2011-05-30 02:55:48 +01:00
parent fff08a9894
commit e7d2e96c23
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/infrastructure/setup/PhabricatorSetup.php
View file

@ -253,8 +253,7 @@ class PhabricatorSetup {
$have_adapter = true;
list($err) = exec_manual('which sendmail');
if ($err) {
if (!Filesystem::pathExists('/usr/bin/sendmail')) {
self::writeFailure();
self::write(
"Setup failure! You don't have a 'sendmail' binary on this system ".