Fix another issue of poisoning too eagerly
authorAlex Crichton <alex@alexcrichton.com>
Fri, 6 Apr 2018 19:35:37 +0000 (12:35 -0700)
committerAlex Crichton <alex@alexcrichton.com>
Fri, 6 Apr 2018 19:37:29 +0000 (12:37 -0700)
commitc7a11056e27080875aa013a8b831418515b5e85a
treec0978f710ca412d5d0dbc0b4821a8127c112bc71
parentd83f1130f36c191763f50ec5373a28e14195f5b5
Fix another issue of poisoning too eagerly

This commit extends the fix in #5288 by moving the logic added farther up in the
loop over package dependencies. This means that we won't recursively look at
optional/dev path dependencies which aren't members of the workspace. This
should fix the new issue that came up in #5257

Closes #5257
src/cargo/ops/resolve.rs
tests/testsuite/freshness.rs