Page MenuHomePhabricator

D9173.id21788.diff
No OneTemporary

D9173.id21788.diff

diff --git a/src/lint/linter/ArcanistJSHintLinter.php b/src/lint/linter/ArcanistJSHintLinter.php
--- a/src/lint/linter/ArcanistJSHintLinter.php
+++ b/src/lint/linter/ArcanistJSHintLinter.php
@@ -49,11 +49,14 @@
}
public function getVersion() {
- list($stdout) = execx('%C --version', $this->getExecutableCommand());
+ // NOTE: `jshint --version` emits version information on stderr, not stdout.
+ list($stdout, $stderr) = execx(
+ '%C --version',
+ $this->getExecutableCommand());
$matches = array();
$regex = '/^jshint v(?P<version>\d+\.\d+\.\d+)$/';
- if (preg_match($regex, $stdout, $matches)) {
+ if (preg_match($regex, $stderr, $matches)) {
return $matches['version'];
} else {
return false;

File Metadata

Mime Type
text/plain
Expires
Fri, Apr 25, 8:21 PM (6 h, 57 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7719948
Default Alt Text
D9173.id21788.diff (762 B)

Event Timeline