don't need the directory here

minor edits to create-disk.sh to work with some of the changes for now

.github/workflows/daily.yaml

@@ -3,7 +3,7 @@ on:
   push:
     branches: [ $default-branch ]
   schedule:
-    - cron: "0 7 * * 1"   # mondays at 07:00
+    - cron: "0 7 * * *"   # daily 07:00
   workflow_dispatch: {}
 
 jobs:

Makefile

@@ -13,74 +13,93 @@
 #     You should have received a copy of the GNU General Public License
 #     along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-include := './include'
-
-mbr_source_files := $(wildcard src/mbr/*.nasm)
-vbr_source_files := $(wildcard src/vbr/*.nasm)
-stage2_source_files := $(wildcard src/stage2/*.nasm)
-boottest_source_files := $(wildcard src/miniboot32/*.nasm)
-
-mbr_binary_files := $(patsubst src/mbr/%.nasm, build/%.bin, $(mbr_source_files))
-vbr_binary_files := $(patsubst src/vbr/%.nasm, build/%.bin, $(vbr_source_files))
-stage2_binary_files := $(patsubst src/stage2/%.nasm, build/%.bin, $(stage2_source_files))
-boottest_binary_files := $(patsubst src/miniboot32/%.nasm, build/%.bin, $(boottest_source_files))
-
-build_dir := 'build'
+INCDIR := include
+BUILD_DIR := build
+IMG := $(BUILD_DIR)/disk.img
+IMGZ := $(BUILD_DIR)/disk.img.gz
+DEP_DIR     := $(BUILD_DIR)/deps
 
 # Get current Git version (tag) and hash
-GIT_VERSION := $(shell git describe --tags)
-GIT_HASH := $(shell git rev-parse HEAD)
+GIT_VERSION := $(shell git describe --tags --always --dirty 2>/dev/null || echo unknown)
+GIT_HASH := $(shell git rev-parse HEAD 2>/dev/null || echo unknown)
 GIT_NASM_DEFINES := -D __GIT_VER__='"$(GIT_VERSION)"' -D __GIT_HASH__='"$(GIT_HASH)"'
 
-iso := 'build/disk.img'
-isoz := 'build/output/disk.img.gz'
+NASMFLAGS   := -Wall -f bin -i$(INCDIR)/ $(GIT_NASM_DEFINES)
 
-qemu_args := -L ./bin/ -bios bios.bin -cpu pentium3 -m 128 -S -s -monitor stdio -nic none
-.PHONY: all mbr vbr stage2 boottest clean run run_bochs iso isoz
+QEMU        ?= qemu-system-i386
+QEMU_OPTS   ?= -L ./bin/ -bios bios.bin -cpu pentium3 -m 64 -S -s -monitor stdio -nic none
 
-all: $(iso) $(isoz) $(mbr_binary_files) $(vbr_binary_files) $(stage2_binary_files)
-mbr: $(mbr_binary_files)
-vbr: $(vbr_binary_files)
-stage2: $(stage2_binary_files)
-boottest: $(boottest_binary_files)
+MBR_SRCS 	  := $(wildcard src/mbr/*.nasm)
+VBR_SRCS 	  := $(wildcard src/vbr/*.nasm)
+STAGE2_SRCS   := $(wildcard src/stage2/*.nasm)
+BOOTTEST_SRCS := $(wildcard src/miniboot32/*.nasm)
 
-clean:
-	@rm -v $(build_dir)/*.bin
-	@rm -v $(build_dir)/*.map
-	@rm -v $(build_dir)/*.img
-	@rm -v $(build_dir)/output/*.img.gz
-	@rm -v $(build_dir)/output/*.tar.gz
+MBR_BINS 	  := $(patsubst src/mbr/%.nasm, $(BUILD_DIR)/mbr/%.bin, $(MBR_SRCS))
+VBR_BINS 	  := $(patsubst src/vbr/%.nasm, $(BUILD_DIR)/vbr/%.bin, $(VBR_SRCS))
+STAGE2_BINS   := $(patsubst src/stage2/%.nasm, $(BUILD_DIR)/stage2/%.bin, $(STAGE2_SRCS))
+BOOTTEST_BINS := $(patsubst src/miniboot32/%.nasm, $(BUILD_DIR)/miniboot32/%.bin, $(BOOTTEST_SRCS))
 
-run: $(iso)
-	@sudo qemu-system-i386 $(qemu_args) -hda $(iso)
+ALL_BINS	  := $(MBR_BINS) $(VBR_BINS) $(STAGE2_BINS) $(BOOTTEST_BINS)
 
-run_bochs: $(iso)
-	@bochs -q
+.DEFAULT_GOAL := all
+.DELETE_ON_ERROR:
 
-iso: $(iso)
-	@file $(iso)
+.PHONY: all mbr vbr stage2 boottest clean run run_qemu run_bochs img imgz help
+all: $(IMG) $(IMGZ) $(ALL_BINS)
+mbr: $(MBR_BINS)
+vbr: $(VBR_BINS)
+stage2: $(STAGE2_BINS)
+boottest: $(BOOTTEST_BINS)
 
-isoz: $(isoz)
-	@file $(isoz)
+# Build Rules
 
-build/%.bin: src/mbr/%.nasm
-	@mkdir -p $(shell dirname $@)
-	@nasm -i$(include) -Wall -f bin $(GIT_NASM_DEFINES) $< -o $@
+$(BUILD_DIR)/mbr/%.bin: src/mbr/%.nasm | $(DEP_DIR)
+	@mkdir -p $(@D)
+	@nasm $(NASMFLAGS) -MD $(DEP_DIR)/$*.d -MT $@ $< -o $@
 
-build/%.bin: src/vbr/%.nasm
-	@mkdir -p $(shell dirname $@)
-	@nasm -i$(include) -Wall -f bin $(GIT_NASM_DEFINES) $< -o $@
+$(BUILD_DIR)/vbr/%.bin: src/vbr/%.nasm | $(DEP_DIR)
+	@mkdir -p $(@D)
+	@nasm $(NASMFLAGS) -MD $(DEP_DIR)/$*.d -MT $@ $< -o $@
 
-build/%.bin: src/stage2/%.nasm
-	@mkdir -p $(shell dirname $@)
-	@nasm -i$(include) -Wall -f bin $(GIT_NASM_DEFINES) $< -o $@
+$(BUILD_DIR)/stage2/%.bin: src/stage2/%.nasm | $(DEP_DIR)
+	@mkdir -p $(@D)
+	@nasm $(NASMFLAGS) -MD $(DEP_DIR)/$*.d -MT $@ $< -o $@
 
-build/%.bin: src/miniboot32/%.nasm
-	@mkdir -p $(shell dirname $@)
-	@nasm -i$(include) -Wall -f bin $(GIT_NASM_DEFINES) $< -o $@
+$(BUILD_DIR)/miniboot32/%.bin: src/miniboot32/%.nasm | $(DEP_DIR)
+	@mkdir -p $(@D)
+	@nasm $(NASMFLAGS) -MD $(DEP_DIR)/$*.d -MT $@ $< -o $@
 
-$(iso): $(mbr_binary_files) $(vbr_binary_files) $(stage2_binary_files) $(boottest_binary_files)
+
+$(DEP_DIR):
+	@mkdir -p $@
+
+# Disk image's
+
+$(IMG): $(ALL_BINS) | scripts/create-disk.sh
 	@scripts/create-disk.sh
 
-$(isoz): $(iso)
-	@gzip -9kc $(iso) > $(isoz)
+$(IMGZ): $(IMG)
+	@gzip -9kc $(IMG) > $(IMGZ)
+
+img: $(IMG)
+	@file $(IMG)
+
+imgz: $(IMGZ)
+	@file $(IMGZ)
+
+# Helpers
+
+run: $(IMG)
+	@$(QEMU) $(QEMU_OPTS) -hda $(IMG)
+
+run_bochs: $(IMG)
+	@bochs -q
+
+clean:
+	@$(RM) -rv $(BUILD_DIR)/
+
+help:
+	@awk '/^[A-Za-z0-9_%-]+:/{print $$1}' $(MAKEFILE_LIST) | sed 's/:$$//' | sort -u
+
+# Include header deps
+-include $(wildcard $(DEP_DIR)/*.d)

scripts/create-disk.sh

@@ -21,10 +21,10 @@ if [ $(id -u) = 0 ]; then
 fi
 
 # paths to bootcode
-mbr_file=build/mbr.bin
-vbr_file=build/vbr.bin
-stage2_file=build/stage2.bin
-boottest_file=build/BOOTi686.bin
+mbr_file=build/mbr/mbr.bin
+vbr_file=build/vbr/vbr.bin
+stage2_file=build/stage2/stage2.bin
+boottest_file=build/miniboot32/BOOTi686.bin
 
 # Disk creation options
 disk_img=build/disk.img
@@ -161,9 +161,14 @@ if [[ "$OSTYPE" == "linux-gnu"* ]]; then
         dd if="$part_img" of="$disk_img" bs=$disk_sector_size seek=$part_start conv=notrunc
 
         echo " *** Outputing disk images will be in ./build/output/* *** "
+        
+        if [ ! -d ./build/output ]; then
+            echo "./build/output does not exist. creating"
+            mkdir -p ./build/output
+        fi
         gzip -9c "$disk_img" > "$disk_img_final"
         gzip -9c "$part_img" > "$part_img_final"
-        tar caf "$artifacts_archive" build/*.bin build/*.map
+        tar caf "$artifacts_archive" --exclude ./build/output ./build/ 
 else
         # Unknown.
         echo "Unknown OS type! Supported build hosts systems are GNU/Linux (& WSL)"