chore: simpler armv7 fluvio-run build config

infinyon · Aug 4, 2023 · 5e4bf46 · 5e4bf46
1 parent 1d959f1
commit 5e4bf46
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/crates/fluvio-run/Cargo.toml b/crates/fluvio-run/Cargo.toml
@@ -31,11 +31,15 @@ tracing-subscriber = { workspace = true }
 
 # regardless of TLS, sc and spu always use openssl_tls for now because we need cert API
 fluvio-future = { workspace = true, features = ["subscriber"] }
-fluvio-spu = { path = "../fluvio-spu" }
+fluvio-sc = { path = "../fluvio-sc" }
 fluvio-extension-common = { workspace = true }
 
 [target.'cfg(not(target_arch = "armv7"))'.dependencies]
 fluvio-sc = { path = "../fluvio-sc", features = ["k8"] }
+fluvio-spu = { path = "../fluvio-spu" }
 
+# armv7 does not support smartengine, this turns off the
+# smartengine feature in fluvio-spu
 [target.'cfg(target_arch = "armv7")'.dependencies]
 fluvio-sc = { path = "../fluvio-sc", default-features = false }
+fluvio-spu = { path = "../fluvio-spu", default-features = false }
diff --git a/makefiles/build.mk b/makefiles/build.mk
@@ -16,13 +16,8 @@ build-cli-minimal: install_rustup_target
 	cargo build --bin fluvio -p fluvio-cli $(RELEASE_FLAG) $(TARGET_FLAG) $(VERBOSE_FLAG) \
 	    --no-default-features --features consumer,producer-file-io
 
-ifeq "$(TARGET_FLAG)" "armv7-unknown-linux-gnueabihf"
-	fluvio_run_extra:="--no-default-features"
-else
-	fluvio_run_extra :=""
-endif
 build-cluster: install_rustup_target
-	cargo build --bin fluvio-run -p fluvio-run $(RELEASE_FLAG) $(TARGET_FLAG) $(VERBOSE_FLAG) $(DEBUG_SMARTMODULE_FLAG) $(fluvio_run_extra)
+	cargo build --bin fluvio-run -p fluvio-run $(RELEASE_FLAG) $(TARGET_FLAG) $(VERBOSE_FLAG) $(DEBUG_SMARTMODULE_FLAG)
 
 build-test:	install_rustup_target
 	cargo build --bin fluvio-test -p fluvio-test $(RELEASE_FLAG) $(TARGET_FLAG) $(VERBOSE_FLAG)