From 7fef5742cf0d0eb11ecb6463e5483718286f4bc4 Mon Sep 17 00:00:00 2001
From: Quinn <quinnthepigeon@proton.me>
Date: Thu, 13 Mar 2025 14:27:11 +0100
Subject: [PATCH] allow makefile to execute the program, changed arguments to
 be environment variable based

---
 makefile | 32 +++++++++++++++++++-------------
 1 file changed, 19 insertions(+), 13 deletions(-)

diff --git a/makefile b/makefile
index 5e73275..23c58b6 100644
--- a/makefile
+++ b/makefile
@@ -6,6 +6,7 @@ STD := c17
 LANG = c
 CFLAGS := $(shell pkg-config --cflags sdl2) -Wall -Wextra -Wpedantic -Wno-pointer-arith
 LDFLAGS := $(shell pkg-config --libs sdl2) -lm
+DEBUG ?= 0
 
 ifeq ($(DEBUG),1)
 CFLAGS += -DDEBUG -fsanitize=address,undefined -g -Og
@@ -35,16 +36,23 @@ define wr_colour
 	@echo -e "\033[$(2)m$(1)\033[0m"
 endef
 
-# sets the variables for the different targets
-linux-x86_64:
-	@$(MAKE) _build ARCH=$@ CFLAGS="$(CFLAGS) -target x86_64-pc-linux-gnu"
-win-x86_64:
-	@$(MAKE) _build ARCH=$@ CFLAGS="$(CFLAGS) -target x86_64-pc-windows-gnu" EXT=".exe"
-web:
-	@$(MAKE) _build ARCH=$@ CC=emcc EXT=".html"
+# set the correct environment variables depending on the platform
+ifeq ($(ARCH),linux-x86_64)
+CFLAGS += -target x86_64-pc-linux-gnu
+else ifeq ($(ARCH),win-x86_64)
+CFLAGS += -target x86_64-pc-windows-gnu
+EXT=.exe
+else ifeq ($(ARCH),web)
+CC=emcc
+EXT=.html
+else
+$(error you must set the ARCH environment variable to either 'linux-x86_64', 'win-x86_64' or 'web')
+endif
 
-all: linux-x86_64 win-x86_64 web
-_build: compile_commands.json $(DIR) $(TARGET) $(ASSETS)
+# compiles and execute the binary
+run: compile
+	./$(TARGET)
+compile: compile_commands.json $(DIR) $(TARGET) $(ASSETS)
 clean:
 	rm -rf $(DIR_BIN) $(DIR_OBJ) compile_commands.json
 
@@ -77,11 +85,9 @@ ifeq ($(DEBUG),1)
 compile_commands.json: makefile
 	$(MAKE) clean
 	@touch compile_commands.json
-	bear -- make $(ARCH)
+	bear -- make compile
 else
-compile_commands.json: makefile
-	$(MAKE) clean
-	@touch compile_commands.json
+compile_commands.json:
 endif
 
 # include the dependencies