From 79b47a76e24c6c7e79617a53185991cf67bb2c0e Mon Sep 17 00:00:00 2001
From: Jef Roosens <roosensjef@gmail.com>
Date: Fri, 18 Feb 2022 21:59:33 +0100
Subject: [PATCH] Remove packages from db before adding new version

---
 src/repo/repo.v | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/src/repo/repo.v b/src/repo/repo.v
index 554b744..1bf2d0c 100644
--- a/src/repo/repo.v
+++ b/src/repo/repo.v
@@ -74,6 +74,9 @@ fn (r &Repo) add(pkg &package.Pkg) ?bool {
 		return false
 	}
 
+	// We remove the older package version first, if present
+	r.remove(pkg.info.name, false) ?
+
 	os.mkdir(pkg_dir) or { return error('Failed to create package directory.') }
 
 	os.write_file(os.join_path_single(pkg_dir, 'desc'), pkg.to_desc()) or {
@@ -92,6 +95,31 @@ fn (r &Repo) add(pkg &package.Pkg) ?bool {
 	return true
 }
 
+// remove removes a package from the database. It returns false if the package
+// wasn't present in the database.
+fn (r &Repo) remove(pkg_name string, sync bool) ?bool {
+	// We iterate over every directory in the repo dir
+	for d in os.ls(r.repo_dir) ? {
+		name := d.split('-')#[..-2].join('-')
+
+		if name == pkg_name {
+			// We lock the mutex here to prevent other routines from creating a
+			// new archive while we removed an entry
+			lock r.mutex {
+				os.rmdir_all(os.join_path_single(r.repo_dir, d)) ?
+			}
+
+			if sync {
+				r.sync() ?
+			}
+
+			return true
+		}
+	}
+
+	return false
+}
+
 // Returns the path where the given package's desc & files files are stored
 fn (r &Repo) pkg_path(pkg &package.Pkg) string {
 	return os.join_path(r.repo_dir, '$pkg.info.name-$pkg.info.version')