fix: not rebuilding for new spigot builds

Author: radstevee

Dockerfile

@@ -36,4 +36,4 @@ COPY . .
 
 RUN cargo install --path .
 
-CMD ["bin_patch_gen"]
+CMD ["bin_patch_gen", "-v", "1.21.3"]

docker-compose.yml

@@ -5,6 +5,7 @@ services:
     volumes:
       - ./config.toml:/app/config.toml
       - ./work:/app/run
+      - /tmp/bpg:/tmp/bin-patch-gen
     mem_limit: 2G
   tests:
     container_name: sploon-bin-patch-gen-tests
@@ -13,4 +14,5 @@ services:
     volumes:
       - ./config.toml:/app/config.toml
       - ./work:/app/run
-    mem_limit: 2G
+      - /tmp/bpg:/tmp/bin-patch-gen
+    mem_limit: 2G

src/lib.rs

@@ -220,7 +220,7 @@ pub async fn run(
                 warn!("{version} metadata is invalid or could not be read! Rebuilding...");
             } else {
                 let patched_meta = patched_meta.unwrap();
-                if remote_meta.refs == patched_meta.commit_hashes && !force_build && library_file.exists() {
+                if remote_meta.refs_eq(patched_meta.commit_hashes) && !force_build && library_file.exists() {
                     info!("Already built version {version}, skipping");
                     continue;
                 }

src/version/schema/spigot.rs

@@ -7,15 +7,26 @@ pub struct SpigotVersionMeta {
     pub refs: SpigotVersionRefs,
 }
 
+impl SpigotVersionMeta {
+    pub fn refs_eq(&self, other: SpigotVersionRefs) -> bool {
+        let refs = self.refs.clone();
+
+        refs.build_data == other.build_data
+            && refs.bukkit == other.bukkit
+            && refs.craft_bukkit == other.craft_bukkit
+            && refs.spigot == other.spigot
+    }
+}
+
 #[serial_pascal]
 pub struct SpigotVersionRefs {
     pub build_data: String,
     pub bukkit: String,
     pub craft_bukkit: String,
-    pub spigot: String
+    pub spigot: String,
 }
 
 #[serial]
 pub struct SpigotBuildData {
-    pub server_url: String
+    pub server_url: String,
 }