search

huonw / travis-cargo

A standalone script that manages running Rust's cargo and several other related features on Travis CI.
https://pypi.python.org/pypi/travis-cargo
Apache License 2.0
174 stars 27 forks source link

Do not install kcov before using it #29

Closed Luthaf closed 9 years ago

Luthaf commented 9 years ago

The -- Installing: ../built/usr/local/share/man/man1/kcov.1 step is taking up to 10 min on my travis builds, and is not needed.

autohuonw commented 9 years ago

Thanks for the pull request, and welcome! You should hear from @huonw (or someone else) soon.

huonw commented 9 years ago

Thanks! Do you have a link to one of the travis builds that are problematic? (Just for my own reference.)

huonw commented 9 years ago

(Included in 0.1.9.)

Luthaf commented 9 years ago

This one for example: https://travis-ci.org/Luthaf/cymbalum/builds/87643392. I created this PR during the installation time!

Luthaf commented 9 years ago

Running the coverage tests take 692.81s (installation of kcov + tests run). Running all the tests in debug mode take 279.16s on my machine (I only run them in release mode on Travis because they are too slow in debug mode).

That is 6'14" for the build + installation of kcov.

huonw commented 9 years ago

Huh, how peculiar; there certainly isn't anything obviously wrong from the log. Hopefully it doesn't reappear.