@enough-analyst-54434: regarding https://github.com/pantsbuild/pants/issues/2057 and your last comment on https://rbcommons.com/s/twitter/r/2677/:

DistributionLocator

no longer depends on

JVM

, but

Ivy

and

Executor

still use

DistributionLocator

without declaring a subsystem dependency (because

Ivy

and

Executor

aren’t tasks or subsystems, so they have no such concept). Should I just require

Distribution

be passed explicitly as an argument all the way down the chain, so that the tasks that ultimately use

Ivy

have to be responsible for declaring a

DistributionLocator

dependency?