Make JvmRun support passthru args.

Review Request #1696 — Created Jan. 30, 2015 and submitted

benjyw
pants
2a5498e...
pants-reviews
ity, zundel
Also add comments to explain an unusual option registration pattern.

Built a jvm binary in a different repo, ran it with passthru args, verified that they were recognized.

CI passes: https://travis-ci.org/pantsbuild/pants/builds/48967731

IT
  1. lgtm!

  2. is that all you have to do to support passthru args now? nice!

    1. That, and actually use them of course...

  3. 
      
BE
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as e08351e577d1cb5110b6df411f42e3a33cd689b5.

ZU
  1. 
      
  2. Thanks for adding the explanation.

  3. 
      
Loading...