For prepare.services, avoid empty classpath entries by only selecting jvm targets that have services defined

Review Request #3058 - Created Oct. 30, 2015 and submitted

Information
Stu Hood
pants
2472
527128f...
Reviewers
pants-reviews
gmalmquist, jsirois, zundel

PrepareServices should only select targets which actually have services. I don't actually know why the previous logic (to select all JvmTargets) existed though, so hoping reviewers have insight there.

This prevents creating empty classpath entries for all JvmTargets (which significantly slows down classloading time).

https://travis-ci.org/pantsbuild/pants/builds/88266699

Eric Ayers
Stu Hood
John Sirois
Stu Hood
Review request changed

Status: Closed (submitted)

Change Summary:

Merged as 8163d6850d0f33d4ea6614a97fc499765e824f94

Loading...