make Docker install valgrind #46

Merged
merged 1 commit into from Nov 23, 2015

Conversation

Projects
None yet
4 participants
Member

edunham commented Nov 23, 2015

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

fixes #45

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

brson was assigned by rust-highfive Nov 23, 2015

r? @brson

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

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

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

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

alexcrichton deleted the unknown repository 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