Plumb fetch timeout through `BinaryUtil`.

Review Request #4196 — Created Aug. 29, 2016 and submitted — Latest diff uploaded

jsirois
pants
jsirois/issues/3817
3817, 3822
6a7e65f...
pants-reviews
gmalmquist, kwlzn, zundel
Previously, the option was stored by `BinaryUtil` but not forwarded to
the underlying `Fetcher` it used.

 src/python/pants/binaries/binary_util.py             |  5 ++++-
 tests/python/pants_test/binaries/BUILD               |  2 ++
 tests/python/pants_test/binaries/test_binary_util.py | 21 +++++++++++++++++----
 3 files changed, 23 insertions(+), 5 deletions(-)

Added failing test that this patch fixes:
./pants test tests/python/pants_test/binaries:binary_util

CI went green here:
https://travis-ci.org/pantsbuild/pants/builds/156068156

Loading...