Page MenuHomePhabricator

Allow ArcanistExternalLinter to specify flags passed to interpreter

Authored by cspeckmim on Jan 20 2016, 6:54 AM.
Referenced Files
Tue, May 21, 10:02 PM
F13235848: D15067.diff
Tue, May 21, 6:53 AM
F13230616: D15067.diff
Mon, May 20, 11:06 PM
F13217727: D15067.diff
Sat, May 18, 7:36 AM
F13210305: D15067.diff
Fri, May 17, 4:42 AM
F13202023: D15067.id36385.diff
Tue, May 14, 8:19 PM
F13199097: D15067.diff
Mon, May 13, 11:40 AM
F13192951: D15067.id36385.diff
Sun, May 12, 10:56 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.