librelist archives

« back to archive

Code review requested: Move "fails_with_llvm" out of def install and into DSL

Code review requested: Move "fails_with_llvm" out of def install and into DSL

From:
Adam Vandenberg
Date:
2011-03-22 @ 01:49
formula provides a method "fails_with_llvm" that can be called from
"def install" in a formula to flag that it won't work with llvm-gcc.
To follow Homebrew style, I'm moving this into the "DSL", like
"depends_on" and "keg_only".

Issue: https://github.com/mxcl/homebrew/issues/4709
Branch: https://github.com/adamv/homebrew/commits/llvm

I've moved this function to the DSL and added a compatibility function
for external formulae.

Any comments, concerns, ets. would be nice, thanks.