From 7aa496efed02bb7cd59866c1d7954799128832a5 Mon Sep 17 00:00:00 2001 From: Max Howell Date: Thu, 30 Aug 2012 10:03:26 -0400 Subject: Fixes #14542; ocaml superenv issues This patch removes most of the settings for CC, CXX etc. because we are trying to be minimal. Then we force the compiler to Homebrew's choice underneath in superenv. We however leave LD because we prefer that build-systems use the c-compiler for linking, it generally works better (copiously tested), however when the build-system explicitly calls ld, we respect that. This gets around the ocaml bug in question, since somehow clang was crashing during link, but the ld tool itself (which is kind of clang, kind of llvm-gcc) is okay with this. Also moved the setting of O (so that cc-args are refurbished) into a make wrapper. Not sure if this matter much, but seems more consistent. --- Library/Formula/objective-caml.rb | 3 --- 1 file changed, 3 deletions(-) (limited to 'Library/Formula') diff --git a/Library/Formula/objective-caml.rb b/Library/Formula/objective-caml.rb index e60343e5d..808d15a80 100644 --- a/Library/Formula/objective-caml.rb +++ b/Library/Formula/objective-caml.rb @@ -9,9 +9,6 @@ class ObjectiveCaml < Formula depends_on :x11 - # Don't strip symbols, so dynamic linking doesn't break. - skip_clean :all - # See http://caml.inria.fr/mantis/view.php?id=5700 def patches "http://caml.inria.fr/mantis/file_download.php?file_id=722&type=bug" -- cgit v1.2.3