From 6c66d0211f58554153084ccb5dcc080d57573290 Mon Sep 17 00:00:00 2001
From: matthew <matthew@sleepy.day>
Date: Fri, 11 Jul 2025 08:15:43 +1000
Subject: [PATCH] build fixes, vulkan debug fix, asset loading, testing
 pipeline creation

---
 assets/shaders/triangle.frag.spv | Bin 0 -> 572 bytes
 assets/shaders/triangle.vert.spv | Bin 0 -> 1512 bytes
 build-vma.sh                     |  24 +++++++
 dub.json                         |   3 +-
 src/gears/main.d                 |  21 +++++++
 src/gears/vulkan.d               |   8 ++-
 src/gears/vulkan_logging.d       |   6 +-
 src/generated/assets_codegen.d   |  24 ++-----
 src/packer/packer.d              |   2 +
 src/shared/alloc.d               |  11 ++++
 src/shared/assets.d              | 103 ++++++++++++++++++++++++++++++-
 11 files changed, 179 insertions(+), 23 deletions(-)
 create mode 100644 assets/shaders/triangle.frag.spv
 create mode 100644 assets/shaders/triangle.vert.spv

diff --git a/assets/shaders/triangle.frag.spv b/assets/shaders/triangle.frag.spv
new file mode 100644
index 0000000000000000000000000000000000000000..c9fb61090d888854d1b7a96b39eb906f3430f92c
GIT binary patch
literal 572
zcmYjNO-sW-6nssZq&D`;g5IR!Q79g?2-12G*-M~`KVT_Il!cg-G!?x01N}i>1ZUPn
zvSs@A&Cc7Ijq98T;&&vJflTFZO~jE7wxRg8s^~eI@2X^eb$v<2P+TjdW+Z`hQHi&W
z$$4-990FtL$OnLnpM&~^p|12rMOv-bDtgfIcB^;QN0#c$<f%?fnZ}iQPpRv*bS96p
zebN?qP_%tL@vL{TukMSiC}9W2vsqrs&1*v{Oflmr1z7Bvx9O)y%B*KLwb=bJ_CB~7
zZ~{Hw#Gd{${P$4#Rmk7gJw3=#-$$jMJ@p|fK859}XU`1eF3?ju1VldX7I*{m(2I+A
ze1q{&mtcAK0O0%+Sk8`W4&|R4*Bm_>-K6Gt|0B{f{}I4>>Uqu_E4cDKJo`b)8{iko
CJunIY

literal 0
HcmV?d00001

diff --git a/assets/shaders/triangle.vert.spv b/assets/shaders/triangle.vert.spv
new file mode 100644
index 0000000000000000000000000000000000000000..291e27e7f3c2d208014be6b535c7ceef4428fae1
GIT binary patch
literal 1512
zcmYk5ZBNud6oqGZ=`OkgA_Bf37EpWvS5zP-MvXyIO*R2T;@4)=tz?qg-E50dKl#u6
z1O6btn0TJHv+0n#_nvdl?c6&vw3=HT({7lqnKOr`T#Ke@8Vb9nUHG7X*6(Hcu(yA(
zhhpBe3L%;WbGe9T^!s&SQ@|D3s%%rXEqf+w>8~mOM@2KL3Ho7hauW29!)QDXv;0RA
zhlx$&aA+rSl-qAHy4jk}rcrV+tP4CT>ic@eEA}3bGMn2`nrS~Y^b(Dd(L{u&nS9~%
zBs`5LpW{g$e-{*6^2Fq-Pl)XldS)$no91Wsr@Eu2%rTd8M~NN3w^=?&qj;L*q{v+)
z$u#HbEI1n${cuLol+Aohhjhd&oZFYl;QWZ*6$h&G<=O9p^2Dre>82dL!r`5c8aVoK
z95Ebc80eQiz>h1;{_8T<l6K`A8~Oru#4C#0^1f_U^&NRIxn<#5d0)obsqiJm)UOHi
za;Smxh1r{W_*WI?Ex}QbpOdj1o-g6=U7ZKtT9BcEcbcn;wca|-q5vG`xNC~3DY<sh
z!;&!HLr<O=uglYin_*cooSR{#%EL1QnD<wo;fAm=ztm5=sJ$r!N8gy^Z&ldk9Dlo}
zB?r%bYcjq?9UA(*BhTL4vAYBAl()eh+?6K}x30*ScQxcN?q)-tnSB{^fj4Dv-1U~c
zFKfuy2YgS)qW-=-Ie6;9+Z9Ltp<?>r4#7QnzD+OSy$W;I)Ps4SXvn{ir=I+w-U4rs
znOSK5it*&%2=S)4TVn2pb9*f{y*-f8*E^}{>5&XQJR17r{{#M$H0-%kadJ~`x8j)j
zR8`FJM240e_srd~53@d%ah{wReSy)tzMd;~eKG64jNakVF!zBBo_fv({7<L;QuZGf
C=4`wG

literal 0
HcmV?d00001

diff --git a/build-vma.sh b/build-vma.sh
index e9dd872..5872b42 100755
--- a/build-vma.sh
+++ b/build-vma.sh
@@ -1,6 +1,30 @@
 #!/bin/bash
 set -eu
 
+# SHADERS
+
+shader_compiler="glslc"
+shader_flags="--target-spv=spv1.6 -std=460"
+shader_out="-oassets/shaders/"
+
+mkdir -p assets/shaders
+
+for shader in src/shaders/*.glsl; do
+	base_name=$(basename -- "$shader" .glsl)
+
+	case "$base_name" in
+		*.vert) shader_stage="-fshader-stage=vert" ;;
+		*.frag) shader_stage="-fshader-stage=frag" ;;
+		*.tesc) shader_stage="-fshader-stage=tesc" ;;
+		*.tese) shader_stage="-fshader-stage=tese" ;;
+		*.geom) shader_stage="-fshader-stage=geom" ;;
+		*.comp) shader_stage="-fshader-stage=comp" ;;
+		*) continue ;;
+	esac
+
+	$shader_compiler $shader_flags $shader_stage $shader "${shader_out}${base_name}.spv"
+done
+
 if 	 [ -x "$(command -v g++)" ]; 		then cpp_compiler="g++"; c_compiler="gcc";
 elif [ -x "$(command -v clang++)" ]; then cpp_compiler="clang++"; c_compiler="clang";
 else echo "Unable to find c++ cpp_compiler"; exit -1; fi;
diff --git a/dub.json b/dub.json
index c27b067..59f9655 100644
--- a/dub.json
+++ b/dub.json
@@ -13,7 +13,7 @@
 			"sourcePaths": ["src/gears", "src/shared", "src/generated"],
 			"libs-linux": ["xcb", "X11", "X11-xcb", "vulkan", "stdc++"],
 			"libs-windows": [],
-			"preGenerateCommands-linux": ["./build-vma.sh"],
+			"preGenerateCommands-linux": ["./build-vma.sh", "build/Packer"],
 			"preGenerateCommands-windows": [],
 		},
 		{
@@ -37,6 +37,7 @@
 			"sourceFiles-linux": ["build/libxxhash.a", "build/libstb_image.a"],
 			"preGenerateCommands-linux": ["./build-vma.sh"],
 			"preGenerateCommands-windows": [],
+			"versions": ["codegen"],
 		}
 	]
 }
diff --git a/src/gears/main.d b/src/gears/main.d
index 1eb6169..2f4ad84 100644
--- a/src/gears/main.d
+++ b/src/gears/main.d
@@ -6,13 +6,34 @@ import u = util : Result;
 import p = platform;
 import a = alloc;
 import vk = vulkan : Vulkan;
+import ap = assets;
+import r = renderer;
 
 void main()
 {
 	p.Window window = p.CreateWindow("Video Game", 1920, 1080);
 
+	assert(ap.OpenAssetPack(), "OpenAssetPack failure");
+
 	Result!(Vulkan) result = vk.Init(&window, u.MB(24), u.MB(32));
 	Vulkan vulkan = result.value;
 
+	u8[] vert_bytes = ap.LoadAssetData("shaders/triangle.vert");
+	u8[] frag_bytes = ap.LoadAssetData("shaders/triangle.frag");
+
+	assert(vert_bytes != null && frag_bytes != null, "Unable to load shader data");
+
+	auto vert_module = vk.BuildShader(&vulkan, vert_bytes);
+	auto frag_module = vk.BuildShader(&vulkan, frag_bytes);
+
+	assert(vert_module.ok && frag_module.ok, "Unable to build vulkan shaders");
+
+	r.GfxPipelineInfo pipeline_info = {
+		vertex_shader: vert_module.value,
+		frag_shader: frag_module.value,
+	};
+
+	auto pipeline = vk.CreateGraphicsPipeline(&vulkan, &pipeline_info);
+
 	vk.Destroy(&vulkan);
 }
diff --git a/src/gears/vulkan.d b/src/gears/vulkan.d
index 21670c8..759ae5a 100644
--- a/src/gears/vulkan.d
+++ b/src/gears/vulkan.d
@@ -313,17 +313,18 @@ CreateGraphicsPipeline(Vulkan* vk, GfxPipelineInfo* build_info)
 		{
 			sType: VK_STRUCTURE_TYPE_PIPELINE_SHADER_STAGE_CREATE_INFO,
 			stage: VK_SHADER_STAGE_VERTEX_BIT,
-			module: cast(VkShaderModule)build_info.vertex_shader,
 			pName: "main",
 		},
 		{
 			sType: VK_STRUCTURE_TYPE_PIPELINE_SHADER_STAGE_CREATE_INFO,
 			stage: VK_SHADER_STAGE_FRAGMENT_BIT,
-			module: cast(VkShaderModule)build_info.frag_shader,
 			pName: "main",
 		},
 	];
 
+	__traits(getMember, shader_info.ptr + 0, "module") = build_info.vertex_shader;
+	__traits(getMember, shader_info.ptr + 1, "module") = build_info.frag_shader;
+
 	VkGraphicsPipelineCreateInfo create_info = {
 		sType: VK_STRUCTURE_TYPE_GRAPHICS_PIPELINE_CREATE_INFO,
 		pNext: &rendering_info,
@@ -341,6 +342,9 @@ CreateGraphicsPipeline(Vulkan* vk, GfxPipelineInfo* build_info)
 	};
 
 	VkPipeline pipeline;
+	
+	VkResult result = vkCreateGraphicsPipelines(vk.device, null, 1, &create_info, null, &pipeline);
+	assert(VkCheck("CreateGraphicsPipeline failure", result), "Unable to build pipeline");
 
 	return pipeline;
 }
diff --git a/src/gears/vulkan_logging.d b/src/gears/vulkan_logging.d
index efd5525..76a1cfc 100644
--- a/src/gears/vulkan_logging.d
+++ b/src/gears/vulkan_logging.d
@@ -1,6 +1,8 @@
 public import includes;
 import std.stdio;
 import vulkan : Vulkan;
+import std.conv;
+import std.string;
 
 VkBool32
 DebugCallback(
@@ -56,8 +58,10 @@ DebugCallback(
 		mt = "Unknown";
 		break;
 	}
+
+	string msg = to!string(callback_data.pMessage);
 	
-	writefln("[%s: %s]\n%s\n", ms, mt, callback_data.pMessage);
+	writefln("[%s: %s]\n%r\n", ms, mt, msg);
 
 	return VK_FALSE;
 }
diff --git a/src/generated/assets_codegen.d b/src/generated/assets_codegen.d
index e3ebbc2..eea100f 100644
--- a/src/generated/assets_codegen.d
+++ b/src/generated/assets_codegen.d
@@ -26,30 +26,18 @@ static immutable u64[] MODEL_HASHES = [
 ];
 
 static immutable string[] SHADER_FILES = [
-	"shaders/gui.frag.spv",
-	"shaders/pbr.frag.spv",
-	"shaders/gui.vert.spv",
-	"shaders/quad.frag.spv",
-	"shaders/quad.vert.spv",
-	"shaders/pbr.vert.spv",
+	"shaders/triangle.vert.spv",
+	"shaders/triangle.frag.spv",
 ];
 
 static immutable string[] SHADER_NAMES = [
-	"shaders/gui.frag",
-	"shaders/pbr.frag",
-	"shaders/gui.vert",
-	"shaders/quad.frag",
-	"shaders/quad.vert",
-	"shaders/pbr.vert",
+	"shaders/triangle.vert",
+	"shaders/triangle.frag",
 ];
 
 static immutable u64[] SHADER_HASHES = [
-	15780387719315455808,
-	2230071466542309169,
-	14797956403837654625,
-	8430018914716708078,
-	14432191255225961360,
-	8518761701216801634,
+	8769314645675479020,
+	6282520872716708711,
 ];
 
 static immutable string[] TEXTURE_FILES = [
diff --git a/src/packer/packer.d b/src/packer/packer.d
index bf6d6c9..85e36e9 100644
--- a/src/packer/packer.d
+++ b/src/packer/packer.d
@@ -149,6 +149,8 @@ TestFile()
 			u8[] data = asset.rawRead(new u8[asset.size()]);
 			assert(data.length == info.length, "TestFile failure: File length read is incorrect");
 
+			assert(hashes[i] == info.hash, "TestFile failure: File hash is incorrect");
+
 			ap.seek(info.offset);
 			u8[] pack_data = ap.rawRead(new u8[info.length]);
 			assert(equal!((a, b) => a == b)(data[], pack_data[]), "TestFile failure: Asset data does not match file data");
diff --git a/src/shared/alloc.d b/src/shared/alloc.d
index 75d3579..e051024 100644
--- a/src/shared/alloc.d
+++ b/src/shared/alloc.d
@@ -93,3 +93,14 @@ Free(Arena* arena)
 {
 	pureFree(arena.mem);
 }
+
+void
+FreeArray(T)(T[] arr)
+{
+	pureFree(arr.ptr);
+}
+
+void Free(T)(T* ptr)
+{
+	pureFree(ptr);
+}
diff --git a/src/shared/assets.d b/src/shared/assets.d
index ffc1985..87af0d8 100644
--- a/src/shared/assets.d
+++ b/src/shared/assets.d
@@ -1,6 +1,30 @@
 import aliases;
 import std.file;
-import assets_codegen;
+import std.stdio;
+import util;
+import std.exception;
+import alloc;
+
+version(codegen) 
+{
+	const u64 ASSET_COUNT = 0;
+
+	enum AssetType { None };
+}
+else 
+{ 
+	import assets_codegen;
+
+	const u64 ASSET_COUNT = MODEL_FILES.length + SHADER_FILES.length + TEXTURE_FILES.length;
+}
+
+File Asset_File;
+
+FileHeader Asset_Header;
+
+AssetInfo[ASSET_COUNT] Asset_Info;
+
+u8[][ASSET_COUNT] Asset_Data;
 
 const u32 FILE_VERSION = 1;
 
@@ -36,3 +60,80 @@ struct AssetInfo
 	u64 length;
 	AssetType type;
 }
+
+bool
+OpenAssetPack()
+{
+	bool success = true;
+	string file_path = isFile("build/assets.sgp") ? "build/assets.sgp" : "assets.sgp";
+
+	// TODO: replace this with something that doesn't throw an exception and figure out if this is the best way to handle thing (probably isnt)
+	try 
+	{
+		Asset_File = File(file_path, "rb");
+	}
+	catch (ErrnoException e)
+	{
+		success = false;
+		Logf("OpenAssetPack failure: Unable to open file %s", file_path);
+	}
+
+	FileHeader[1] header_arr;
+
+	Asset_File.rawRead(header_arr);
+
+	Asset_Header = header_arr[0];
+
+	assert(Asset_Header.file_version == FILE_VERSION, "OpenAssetPack failure: file version incorrect");
+
+	Asset_File.seek(Asset_Header.asset_info_offset);
+
+	Asset_File.rawRead(Asset_Info);
+
+	return success;
+}
+
+u8[]
+LoadAssetData(string name)
+{
+	u64 hash = Hash(name);
+	u8[] data = null;
+
+	foreach(i, info; Asset_Info)
+	{
+		if (info.hash == hash)
+		{
+			if (Asset_Data[i].ptr == null)
+			{
+				Asset_Data[i] = AllocArray!(u8)(info.length);
+				Asset_File.seek(info.offset);
+				Asset_File.rawRead(Asset_Data[i]);
+				assert(Asset_Data[i] != null && Asset_Data[i].length == info.length, "LoadAssetData failure: Asset data loaded incorrectly.");
+			}
+
+			data = Asset_Data[i];
+			break;
+		}
+	}
+
+	return data;
+}
+
+void
+UnloadAssetData(string name)
+{
+	u64 hash = Hash(name);
+
+	foreach(i, info; Asset_Info)
+	{
+		if (info.hash == hash)
+		{
+			if (Asset_Data[i] != null)
+			{
+				FreeArray(Asset_Data[i]);
+				break;
+			}
+		}
+	}
+}
+