From 5f7b90c1a33c3248a7d6cf0dede86e2a24ee2a28 Mon Sep 17 00:00:00 2001
From: Ian Meyer <k@imeyer.io>
Date: Tue, 17 Sep 2024 22:16:27 +0000
Subject: [PATCH] make: Introduce run-binary and release targets

- Now there are multiple ways to run tdiscuss for iterative development!
- `make build` does a development build
- `make release` handles the versioned build

Change-Id: I18439f60e8a4d21858a6ea97ea373cc4e6796e40
Signed-off-by: Ian Meyer <k@imeyer.io>
---
 Makefile | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/Makefile b/Makefile
index 3326ba6..2247807 100644
--- a/Makefile
+++ b/Makefile
@@ -23,15 +23,16 @@ TARGET := tdiscuss-$(PLATFORM)-$(ARCH)
 
 # Bazel build command
 BAZEL := bazelisk
-BAZEL_BUILD_ARGS := build --config=silent --stamp --workspace_status_command="$${PWD}/status.sh"
+BAZEL_BUILD_ARGS := build --config=silent --workspace_status_command="$${PWD}/status.sh"
+BAZEL_RELEASE_ARGS := build --config=silent --stamp --workspace_status_command="$${PWD}/status.sh"
 BAZEL_TEST_ARGS := test --config=silent --build_tests_only --test_output=errors
 BAZEL_RUN_ARGS := run
 # Change the hostname to anything you wish to use for testing
-BAZEL_RUN_TRAILING_ARGS := -- -hostname discuss-dev
+BAZEL_RUN_TRAILING_ARGS := -hostname discuss-dev
 
 .PHONY: all clean
 
-all: build
+all: test build
 
 build:
 	@echo "Building for $(PLATFORM)-$(ARCH)"
@@ -42,8 +43,16 @@ test:
 	$(BAZEL) $(BAZEL_TEST_ARGS) //...
 
 run:
-	@echo "Running for $(PLATFORM)-$(ARCH)"
-	$(BAZEL) $(BAZEL_RUN_ARGS) //:$(TARGET) $(BAZEL_RUN_TRAILING_ARGS)
+	@echo "Running for $(PLATFORM)-$(ARCH) from $(BAZEL)"
+	$(BAZEL) $(BAZEL_RUN_ARGS) //:$(TARGET) -- $(BAZEL_RUN_TRAILING_ARGS)
+
+run-binary:
+	@echo "Running for $(PLATFORM)-$(ARCH) from $(shell $(BAZEL) info bazel-bin)"
+	$(shell $(BAZEL) info bazel-bin)/$(TARGET)_/$(TARGET) $(BAZEL_RUN_TRAILING_ARGS)
+
+release:
+	@echo "Building release for $(PLATFORM)-$(ARCH)"
+	$(BAZEL) $(BAZEL_RELEASE_ARGS) //:$(TARGET)
 
 clean:
 	$(BAZEL) clean