pantsbuild / pants

65%

master: 0%

LAST BUILD ON BRANCH tdesai/twitter-release-fix

branch: tdesai/twitter-release-fix

CHANGE BRANCH

x

Reset

• tdesai/twitter-release-fix
• 2.0.x
• 2.1.x
• 2.2.x
• 2.3.x
• add_transitive_jar_deps
• areitz/add_publish_plugins
• areitz/fix_publish_dup_options2
• cyclic-dep
• dogfood-caching
• dturner/allow-exclude-in-globs
• dturner/fix-bad-random
• dturner/fixit-resource-mapping-fmt
• filedeps_v2engine
• git-root
• init_goal
• lahosken/docsitegen_begin
• master
• options
• release_0.0.26
• release_0.0.28
• release_0.0.29
• release_0.0.30
• release_0.0.31
• release_0.0.32
• release_0.0.33
• release_0.0.34
• release_0.0.35
• release_0.0.36
• release_0.0.37
• release_0.0.38
• release_0.0.40
• release_0.0.41
• release_0.0.42
• release_0.0.43
• release_0.0.44
• release_0.0.45
• release_0.0.46
• release_0.0.47
• release_0.0.48
• release_1.7.0rc0
• release_2.0.0
• release_2.0.0.dev4
• release_2.0.0.dev5
• release_2.0.0.dev6
• release_2.0.0.dev7
• release_2.0.0.dev8
• release_2.0.0.dev9
• release_2.0.0a0
• release_2.0.0a1
• release_2.0.0a2
• release_2.0.0a3
• release_2.0.0b1
• release_2.0.0b2
• release_2.0.0b3
• release_2.0.0rc0
• release_2.0.0rc1
• release_2.0.0rc2
• release_2.0.0rc3
• release_2.0.1
• release_2.0.1rc0
• release_2.0.1rc1
• release_2.0.1rc2
• release_2.0.1rc3
• release_2.0.1rc4
• release_2.0.2
• release_2.0.2rc0
• release_2.1.0
• release_2.1.0.dev0
• release_2.1.0rc0
• release_2.1.0rc1
• release_2.1.0rc2
• release_2.1.0rc3
• release_2.1.1
• release_2.1.1rc0
• release_2.1.1rc1
• release_2.1.1rc2
• release_2.1.2
• release_2.1.2rc0
• release_2.2.0
• release_2.2.0.dev1
• release_2.2.0rc0
• release_2.2.0rc1
• release_2.2.0rc2
• release_2.2.1
• release_2.2.1rc0
• release_2.2.1rc1
• release_2.2.1rc2
• release_2.2.2
• release_2.2.2rc0
• release_2.2.2rc1
• release_2.2.3
• release_2.2.3rc0
• release_2.2.4
• release_2.2.4rc0
• release_2.3.0
• release_2.3.0.dev0
• release_2.3.0.dev1
• release_2.3.0.dev2
• release_2.3.0.dev3
• release_2.3.0rc0
• release_2.3.0rc1
• release_2.3.0rc2
• release_2.3.0rc3
• release_2.3.0rc4
• release_2.3.1
• release_2.3.1rc0
• release_2.3.1rc2
• release_2.3.1rc3
• release_2.3.2
• release_2.3.2rc0
• release_2.3.3
• release_2.3.3rc0
• release_2.4.0.dev0
• release_2.4.0.dev1
• scala-version
• show
• stuhood/1396-and-1395
• stuhood/1396-and-1395-with-redundant-resources-fix
• stuhood/1396-and-1400-integration
• stuhood/avoid-reparsing-analysis
• stuhood/deep-analysis-validation
• stuhood/invalidate-bad-cached-analysis
• stuhood/isolation-integration-three
• stuhood/isolation-integration-two
• stuhood/jvm-compile-isolated-strategy
• stuhood/missing-product-debug
• stuhood/no-classes-for-target
• stuhood/pants-1.2.4-sans-process-edits
• stuhood/perf-integration
• stuhood/prep-2.2.0.dev2
• stuhood/rewind
• stuhood/skip-missing-depfile
• stuhood/switch-classpath-to-union-product
• stuhood/test-populate-products
• stuhood/zinc-in-platform-version
• tc-auth-plugin
• tdesai/add_target_type_to_depmap
• tdesai/feature_javasources
• tdesai/fix_pants_script_in_non_dev_mode
• tdesai/fix_perf_issue
• tdesai/pants_plugin_integrationtest
• tdesai/test_scalac_compiler_issues
• test-remote-cache
• twitter/pants-1.2.3
• twitter/pants-1.2.4
• twitter/pants-1.2.6
• zundel/doc-fork-and-ci
• zundel/fixup-changelog
• zundel/javadoc-fixes
• zundel/update-junit-runner-0.0.8
• zundel/zinc-error-output-repro

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/