2
0%
main: 93%

Ran 11 Oct 2017 04:29AM UTC

Files 63

Run time 3s

Badge

Embed ▾

Committed 30 Nov 2015 08:42PM UTC coverage: 23.086%. First build

Job # CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'"

Build Type

push

travis-ci

Committed by

ericzundel

Commit Message

Update the dependencies returned from ivy to be in a stable (sorted) order.

Reproduces a root cause of slow compiles noticed at Square.

The resolution report from Ivy doesn't return the deps from a target in a stable order. In one case, we get the dependencies of the `jetty-jsp` dep which lists jetty-schema before servlet-api.  In another case it reverses the order of those two deps.

This causes the classpath to be ordered differently when compiling different targets, causing unecessary cache invalidation, and possibly different results when compiling transitive dependencies.  To resolve this, we will sort dependencies alphabetically to insure a stable ordering.

Testing Done:
CI is green at https://travis-ci.org/pantsbuild/pants/builds/93252409

Tried this in our local build. Its an improvement in the number of transitive deps that need to be recompiled when 2 targets share a lot of commone deps.  In one instance, it prevented 67 targets from being recompiled and decreased build time by 4 minutes (was 9 minutes)

Bugs closed: 2607, 2618

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

Coverage Stats

1631 of 7065 relevant lines covered (23.09%)

0.23 hits per line

pantsbuild / pants / 8110 / 2
0%
main: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8110.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'")

pantsbuild / pants / 8110 / 2 0% main: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8110.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'")

pantsbuild / pants / 8110 / 2
0%
main: 93%

README BADGES
x