[coverage] Fixing instrumentation classpath mutation to support multiple targets and entries.

Review Request #3108 - Created Nov. 11, 2015 and submitted

Information
Justin Trobec
pants
jtrobec/coverage/fix-inst-classpaths
2540
Reviewers
pants-reviews
stuhood

This is the 2nd of 2 branches to fix issues with cobertura integration that became evident against non-trivial targets. It modifies the runtime classpath mutation into the instrumentation classpath so that we correctly handle multiple targets and entries per target.

Manually tested against non-trivial targets in the twitter code-base, and smaller targets in the oss pants codebase.

Issues

  • 0
  • 2
  • 0
  • 2
Description From Last Updated
Luc Cary
Justin Trobec
Stu Hood
Stu Hood
Stu Hood
Justin Trobec
Review request changed

Status: Closed (submitted)

Change Summary:

Merged as 80ab54273ad5bc35f915cea36d99d52a04f1d5f6

Loading...