More gentle refactoring of IvyUtils.

Review Request #1646 - Created Jan. 23, 2015 and submitted

Information
Benjy Weinberger
pants
964b4e9...
Reviewers
pants-reviews
ity, jsirois, zundel
- Moved ivy execution logic into the mixin, allowing its interface to
be simplified.
- An Ivy instance no longer encapsulates an executor. Clients can pass
an executor into Ivy.execute() instead.
- Don't bootstrap Ivy with the executor intended for it to run in after
bootstrapping.  Use a subprocess executor instead, like for all other
bootstrapping.  Allows us to not pass an executor around quite so much.

This is another step on the way to reworking the ivy/jvmtask/nailgun triad of
confusion.

CI passes: https://travis-ci.org/pantsbuild/pants/builds/47984451

Eric Ayers
Eric Ayers
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 47e9140e9d1e0da89a47fc84892b4508c575e051.

Ity Kaul
Ship It!
Loading...