Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Commit

Permalink
botan: fix xcode-only install
Browse files Browse the repository at this point in the history
- botan's configure.py ignores our CFLAGS and switches the compiler to
  llvm :-/ We can use the MACH_OPT variable to insert our CLFAGS and
  make this build with Xcode-only setups.
- Using ENV.compiler.to_s for the compiler string.
- Fixing the "ld: unknown option soname" issue when using clang.

Closes #13065.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
  • Loading branch information
samueljohn authored and jacknagel committed Jul 20, 2012
1 parent 42c62f7 commit 89b6037
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion Library/Formula/botan.rb
Expand Up @@ -13,8 +13,14 @@ def install
args = ["--prefix=#{prefix}"] args = ["--prefix=#{prefix}"]
args << "--cpu=x86_64" if MacOS.prefer_64_bit? args << "--cpu=x86_64" if MacOS.prefer_64_bit?
args << "--enable-debug" if ARGV.include? "--enable-debug" args << "--enable-debug" if ARGV.include? "--enable-debug"
# The --cc option needs "clang" or "gcc" and not the full path.
args << "--cc=#{ENV.compiler.to_s}"


system "./configure.py", *args system "./configure.py", *args
system "make install" # "-soname" can lead to issues like https://github.com/mxcl/homebrew/issues/11972
inreplace "Makefile", "-Wl,-soname", "-dynamiclib -install_name "
# A hack to force them use our CFLAGS. MACH_OPT is empty in the Makefile
# but used for each call to cc/ld.
system "make", "install", "MACH_OPT=#{ENV.cflags}"
end end
end end

0 comments on commit 89b6037

Please sign in to comment.