From fb88bb9da69e721f3e91f4dfc1e42e8b7b7f9df3 Mon Sep 17 00:00:00 2001 From: Phil Dibowitz Date: Tue, 16 Apr 2013 15:09:55 -0700 Subject: [PATCH] Add a configuration for python path Test Plan: not a clue Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5717 --- src/lint/linter/ArcanistPEP8Linter.php | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/lint/linter/ArcanistPEP8Linter.php b/src/lint/linter/ArcanistPEP8Linter.php index 215a8a43..346fd2ef 100644 --- a/src/lint/linter/ArcanistPEP8Linter.php +++ b/src/lint/linter/ArcanistPEP8Linter.php @@ -41,7 +41,7 @@ final class ArcanistPEP8Linter extends ArcanistFutureLinter { $bin = $working_copy->getConfig('lint.pep8.bin'); 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'). '/../externals/pep8/pep8.py'); } 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; }