revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-26 08:42:40 +01:00
Code Issues Releases Wiki Activity

Add a configuration for python path

Browse source 
Test Plan: not a clue

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D5717
This commit is contained in:
Phil Dibowitz 2013-04-16 15:09:55 -07:00
parent b3108661bb
commit fb88bb9da6
1 changed files with 1 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/lint/linter/ArcanistPEP8Linter.php
View file

@ -41,7 +41,7 @@ final class ArcanistPEP8Linter extends ArcanistFutureLinter {
$bin = $working_copy->getConfig('lint.pep8.bin'); $bin = $working_copy->getConfig('lint.pep8.bin');
if ($bin === null && $prefix === null) { if ($bin === null && $prefix === null) {
$bin = csprintf('/usr/bin/env python %s', $bin = csprintf('/usr/bin/env python2.6 %s',
phutil_get_library_root('arcanist'). phutil_get_library_root('arcanist').
'/../externals/pep8/pep8.py'); '/../externals/pep8/pep8.py');
} else { } else {
@ -74,13 +74,6 @@ final class ArcanistPEP8Linter extends ArcanistFutureLinter {
} }
} }
list(, $stderr) = execx('/usr/bin/env python -V');
if ($stderr < 'Python 2.5') {
throw new ArcanistUsageException(
"Python 2.5 or greater is required to run the PEP8 Python linter, but ".
rtrim($stderr)." was found instead.");
}
return $bin; return $bin;
} }