Skip to content

Commit

Permalink
wasi-runtimes: reduce :build dependencies
Browse files Browse the repository at this point in the history
We don't need the linkers at build-time if we set
`CMAKE_TRY_COMPILE_TARGET_TYPE` to `STATIC_LIBRARY`.
  • Loading branch information
carlocab committed Nov 15, 2024
1 parent 5d7319c commit 6b058b7
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions Formula/w/wasi-runtimes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,9 @@ class WasiRuntimes < Formula
end

depends_on "cmake" => :build
depends_on "lld" => [:build, :test]
depends_on "wasi-libc" => [:build, :test]
depends_on "wasm-component-ld" => [:build, :test]
depends_on "lld" => :test
depends_on "wasm-component-ld" => :test
depends_on "wasmtime" => :test
depends_on "llvm"

Expand Down Expand Up @@ -55,6 +55,7 @@ def install
-DCMAKE_C_COMPILER_WORKS=ON
-DCMAKE_CXX_COMPILER_WORKS=ON
-DCMAKE_SYSROOT=#{wasi_libc.opt_share}/wasi-sysroot
-DCMAKE_TRY_COMPILE_TARGET_TYPE=STATIC_LIBRARY
-DCMAKE_FIND_FRAMEWORK=NEVER
-DCMAKE_VERBOSE_MAKEFILE=ON
-DCMAKE_PROJECT_TOP_LEVEL_INCLUDES=#{HOMEBREW_LIBRARY_PATH}/cmake/trap_fetchcontent_provider.cmake
Expand Down

0 comments on commit 6b058b7

Please sign in to comment.