{"push_id":838959734,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"2de18d5c0ae1c26fcaa14fabec08399d5de2c781","before":"94d50d9eff739010c4aac04856db67164834513c","commits":[{"sha":"daf33dbf6e4dec679826e0486344bd6fb51758af","author":{"email":"bb1f132dcb86bfc59bf4a043a3139a9ac7c0b658@gmail.com","name":"Stu Hood"},"message":"Handle case where only transitive dependencies have changed during an incremental build\n\nBecause of [#2446](https://github.com/pantsbuild/pants/issues/2446), when you add and remove a dep from a target in succession for a task that uses transitive invalidation, it's possible for an incremental compile to occur in the same `results_dir` as the previous attempt. In that case, we were seeing an error (rightly, as it was an unexpected case) because the previous `results_dir` had not been initialized.\n\n- Work around the bug by always initializing `previous_results_dir`, even if it would be the same as the current `results_dir`\n- Add a test that doesn't quite reproduce the issue, and expect it to fail\n\nTesting Done:\nFixes:\n\n    compile(src/scala/org/pantsbuild/zinc:zinc) failed: There is no previous_results_dir for: VT(src.scala.org.pantsbuild.zinc.zinc, invalid)\n\nexpected to go green:\nhttps://travis-ci.org/pantsbuild/pants/builds/87097649\n\nBugs closed: 2447\n\nReviewed at https://rbcommons.com/s/twitter/r/3028/","distinct":true,"url":"https://api.github.com/repos/pantsbuild-osx/pants/commits/daf33dbf6e4dec679826e0486344bd6fb51758af"},{"sha":"2de18d5c0ae1c26fcaa14fabec08399d5de2c781","author":{"email":"89eb73fbef71259ee15b5c1414c7a16ad9206fb5@googlegroups.com","name":"Pants CI"},"message":"Sync of https://github.com/pantsbuild/pants/commit/daf33dbf6e4dec679826e0486344bd6fb51758af","distinct":true,"url":"https://api.github.com/repos/pantsbuild-osx/pants/commits/2de18d5c0ae1c26fcaa14fabec08399d5de2c781"}]}