From 25855c4427cc49db63c0f74b29ede2fe0993179d Mon Sep 17 00:00:00 2001
From: Joshua Spence <josh@joshuaspence.com>
Date: Wed, 20 May 2015 07:04:07 +1000
Subject: [PATCH] Don't explicitly use an interpreter for `pep8`

Summary: Depends on D9430. If we are using the system installation of `pep8`, then it probably unnecessary (and possibly wrong) to specify `python2.6` as the default interpreter.

Test Plan: Ran `arc unit`.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: vrusinov, epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D9474
---
 src/lint/linter/ArcanistPEP8Linter.php | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/lint/linter/ArcanistPEP8Linter.php b/src/lint/linter/ArcanistPEP8Linter.php
index 99a11101..cdf107e2 100644
--- a/src/lint/linter/ArcanistPEP8Linter.php
+++ b/src/lint/linter/ArcanistPEP8Linter.php
@@ -31,14 +31,6 @@ final class ArcanistPEP8Linter extends ArcanistExternalLinter {
     return $this->getDeprecatedConfiguration('lint.pep8.options', array());
   }
 
-  public function shouldUseInterpreter() {
-    return ($this->getDefaultBinary() !== 'pep8');
-  }
-
-  public function getDefaultInterpreter() {
-    return 'python2.6';
-  }
-
   public function getDefaultBinary() {
     $prefix = $this->getDeprecatedConfiguration('lint.pep8.prefix');
     $bin = $this->getDeprecatedConfiguration('lint.pep8.bin', 'pep8');