make Docker install valgrind #46

Merged
merged 1 commit into from Nov 23, 2015

Conversation

Projects
None yet
4 participants
Contributor

edunham commented Nov 23, 2015

Checked for a valgrind --version on prod-slave-linux-3 but it's not
installed there.

fixes rust-lang#45

Instruct Docker to install valgrind
Checked for a `valgrind --version` on prod-slave-linux-3 but it's not
installed there.

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

brson added a commit that referenced this pull request Nov 23, 2015

Merge pull request #46 from edunham/valgrind
make Docker install valgrind

@brson brson merged commit ccdf530 into rust-lang-deprecated:master Nov 23, 2015

@alexcrichton alexcrichton deleted the edunham:valgrind branch Nov 23, 2015

Owner

alexcrichton commented Nov 23, 2015

We'll want to ensure this works ok in dev on 32 and 64 bit as I think hokiness was required in the past to get a 32-bit valgrind working. I've pushed this with the tag 2015-11-23 as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment