lurk-lab / circom-scotia

Other
18 stars 9 forks source link

Ci upheaval #14

Closed huitseeker closed 11 months ago

huitseeker commented 11 months ago

Re-do of #13

winston-h-zhang commented 11 months ago

Could you fix the LLVM cargo dependencies for CI? I think this should work, but untested

diff --git a/Cargo.toml b/Cargo.toml
index e651aeb..e25eb8d 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -29,9 +29,7 @@ thiserror = "1.0.43"

 [target.'cfg(not(target_arch = "wasm32"))'.dependencies]
 wasmer = "4.0.0"
-
-[target.'cfg(all(not(target_arch = "wasm32"), features = "llvm"))'.dependencies]
-wasmer-compiler-llvm = "4.1.1"
+wasmer-compiler-llvm = { version = "4.1.1", optional = true }

 [target.'cfg(target_arch = "wasm32")'.dependencies]
 wasmer = { version = "4.0.0", features = ["std", "js"], default-features = false }
@@ -40,4 +38,4 @@ getrandom = { version = "0.2.10", features = ["js"] }
 [features]
 default = ["circom-2"]
 circom-2 = []
-llvm = []
+llvm = ["dep:wasmer-compiler-llvm"]