fix(build): use different flags per platform

zama-ai · Oct 20, 2023 · 60412d6 · 60412d6
1 parent 12086c6
commit 60412d6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/Makefile b/Makefile
@@ -1,19 +1,14 @@
-ifeq ($(shell uname), Darwin)
-	CGO_LDFLAGS := -framework Security
-endif
-
 .PHONY: build
 build: build-tfhe-rs-capi
-	cd fhevm && CGO_LDFLAGS='$(CGO_LDFLAGS)' go build .
+	cd fhevm && go build .
 
 .PHONY: test
 test: build-tfhe-rs-capi
-	cd fhevm && CGO_LDFLAGS='$(CGO_LDFLAGS)' go test -v .
+	cd fhevm && go test -v .
 
 .PHONY: build-tfhe-rs-capi
 build-tfhe-rs-capi:
-	cd tfhe-rs && make build_c_api_experimental_deterministic_fft \
-	&& cd target/release && rm -f *.dylib *.dll *.so
+	cd tfhe-rs && make build_c_api_experimental_deterministic_fft
 
 .PHONY: clean
 clean:

diff --git a/fhevm/tfhe.go b/fhevm/tfhe.go
@@ -17,8 +17,10 @@
 package fhevm
 
 /*
-#cgo CFLAGS: -O3 -I../tfhe-rs/target/release
-#cgo LDFLAGS: -L../tfhe-rs/target/release -ltfhe -lm
+#cgo linux CFLAGS: -O3 -I../tfhe-rs/target/release
+#cgo linux LDFLAGS: -L../tfhe-rs/target/release -l:libtfhe.a -lm
+#cgo darwin CFLAGS: -O3 -I../tfhe-rs/target/release ../tfhe-rs/target/release/libtfhe.a
+#cgo darwin LDFLAGS: -framework Security -lm
 
 #include <tfhe.h>