More gentle refactoring of IvyUtils.

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

benjyw
pants
964b4e9...
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

ZU
  1. 
      
  2. Can I trust rbcommons: is this just copied or is there some difference I should pay closer attention to?

    1. It's copied, and the executor argument added.

  3. 
      
ZU
  1. Ship It!
  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 47e9140e9d1e0da89a47fc84892b4508c575e051.

IT
  1. Ship It!
  2. 
      
Loading...