Fix HEAD installation of valgrind. #8040

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

RuiPereira commented Oct 9, 2011

Current Valgrind HEAD now supports Lion.

@RuiPereira RuiPereira Fix HEAD installation of valgrind.
Current Valgrind HEAD now supports Lion.
25bb75a
Contributor

dominickng commented Oct 10, 2011

This patch requires --force to get it to install on Lion. Other than that, it seems to work fine for me for C and C++ programs. Most impressively, I tested a monster of a C++ program that uses its own memory pools and overloaded memory allocators (using ~500MB of memory at peak), and it seemed to run and give the same leak results as on Linux.

Tested on OS X 10.7.1 on a 2011 MacBook Air.

Contributor

RuiPereira commented Oct 11, 2011

Right, I haven't touched the --force part, but it's true that we could ignore that in ARGV.build_head cases.

Contributor

Sharpie commented Oct 13, 2011

Looks good, I'm building and testing Valgrind right now. I'll probably pull the patch and make the formula fail on Lion unless --HEAD is passed.

Sharpie closed this in 358ff1e Oct 13, 2011

Contributor

Sharpie commented Oct 13, 2011

Thanks.

@Sharpie Sharpie added a commit to Sharpie/homebrew that referenced this pull request Sep 12, 2012

@RuiPereira @Sharpie RuiPereira + Sharpie Valgrind: HEAD installs now work on Lion
Closes #8040.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
a87808f

@snakeyroc3 snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request Dec 17, 2012

@RuiPereira @Sharpie RuiPereira + Sharpie Valgrind: HEAD installs now work on Lion
Closes #8040.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
953b078

xu-cheng locked and limited conversation to collaborators Feb 16, 2016

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