From 9bae517a3831bfe3594664b0c7d9dbb0f1b477c7 Mon Sep 17 00:00:00 2001 From: epriestley Date: Sun, 29 Sep 2013 07:44:33 -0700 Subject: [PATCH] Fix a stiring issue in the External linter Summary: See error message in D7170 -- this should be a `.`, not a `,`. Test Plan: Faked interprerter and got reasonable error message: > Unable to locate interpreter "TESTpython2.6" to run linter ArcanistPEP8Linter. You may need to install the intepreter, or adjust your linter configuration. > TO INSTALL: Install PEP8 using `easy_install pep8`. This doesn't fix the //real// error, which is that the test should skip if you don't have the interpreter/binary, but that's a little more involved. Reviewers: hach-que, chad, btrahan Reviewed By: chad CC: aran Differential Revision: https://secure.phabricator.com/D7172 --- src/lint/linter/ArcanistExternalLinter.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lint/linter/ArcanistExternalLinter.php b/src/lint/linter/ArcanistExternalLinter.php index b7031b66..d8fde402 100644 --- a/src/lint/linter/ArcanistExternalLinter.php +++ b/src/lint/linter/ArcanistExternalLinter.php @@ -285,7 +285,7 @@ abstract class ArcanistExternalLinter extends ArcanistFutureLinter { pht( 'Unable to locate interpreter "%s" to run linter %s. You may '. 'need to install the intepreter, or adjust your linter '. - 'configuration.', + 'configuration.'. "\nTO INSTALL: %s", $interpreter, get_class($this),