pantsbuild / pants / 3136

65%

main: 93%

push

travis-ci

CmdLineProcessor uses binary class name (http://docs.oracle.com/javase/specs/jvms/se7/html/jvms-4.html#jvms-4.2.1),

https://github.com/twitter/commons/blob/master/src/java/com/twitter/common/args/apt/CmdLineProcessor.java#L653

Classes that uses CmdLineProcessor as annotation processor would fail,

"Expected classname -> path, got "com.twitter.common.args.ArgsTest$App -> ..."

This patch adds $ to the class name matching, and changed the format of compile/jvm/java/classes/deprecation_report.txt slightly so it's easier to load in testing

Testing Done:
https://travis-ci.org/pantsbuild/pants/builds/44126276 passed

Bugs closed: 868

Reviewed at https://rbcommons.com/s/twitter/r/1489/

1 of 1 new or added line in 1 file covered. (100.0%)

11913 of 18345 relevant lines covered (64.94%)

0.65 hits per line