Page MenuHomePhabricator

Allow ArcanistExternalLinter to specify flags passed to interpreter

Authored by cspeckmim on Jan 20 2016, 6:54 AM.
Referenced Files
Tue, Jun 25, 3:56 PM
F13361741: D15067.id36385.diff
Tue, Jun 25, 3:29 PM
F13352030: D15067.diff
Sun, Jun 23, 11:16 AM
F13348218: D15067.id36385.diff
Sat, Jun 22, 6:27 PM
F13346007: D15067.diff
Fri, Jun 21, 12:44 PM
F13342170: D15067.id36385.diff
Thu, Jun 20, 10:38 AM
F13334657: D15067.diff
Wed, Jun 19, 12:25 AM
F13320387: D15067.diff
Fri, Jun 14, 5:11 AM


Group Reviewers
Blessed Reviewers

Currently linters are only capable of specifying flags to pass to the binary but not to the interpreter. This change follows the existing framework for specifying binary flags to allow for passing flags to the interpreter.

NOTE: This change is extracted from D14632
Test Plan


Diff Detail

rARC Arcanist
Lint Passed
Tests Passed
Build Status
Buildable 10220
Build 12422: arc lint + arc unit

Event Timeline

cspeckmim retitled this revision from to Allow ArcanistExternalLinter to specify flags passed to interpreter.
cspeckmim updated this object.
cspeckmim edited the test plan for this revision. (Show Details)
cspeckmim added a subscriber: avivey.

In the least the ECO form probably needs filled with more detail

@avivey Is there a chance to get this change landed? It seems to be unobtrusive and is needed to have the Java lint'ers in D14632 live outside of upstream as an extension. I can imagine there might be some other lint implementations which require this functionality as well. Let me know if you need any changes, I'm more then willing to make them.

I don't really know - looks like @cspeckmim isn't completely happy with this yet.

A possible work-around is to embed a script in the library that will invoke the jar - I'll write something on the other side.

I pulled this out from D14632 as-is. The only thing I think this needs is possibly UT (though similar functionality doesn't appear to be covered by UT already). I haven't moved on this in a while as I don't have any expectation for this going upstream due to the forward-path indicated by @epriestley.