Release 2.14.2

This reverts commit d30d512eb6.

docs/ahriman-architecture.svg

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" standalone="no"?>
 <!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN"
  "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
-<!-- Generated by graphviz version 12.1.0 (0)
+<!-- Generated by graphviz version 12.1.1 (0)
  -->
 <!-- Title: G Pages: 1 -->
 <svg width="28485pt" height="4761pt"

package/archlinux/PKGBUILD

@@ -1,7 +1,7 @@
 # Maintainer: Evgeniy Alekseev
 
 pkgname='ahriman'
-pkgver=2.14.1
+pkgver=2.14.2
 pkgrel=1
 pkgdesc="ArcH linux ReposItory MANager"
 arch=('any')

package/share/man/man1/ahriman.1

@@ -1,4 +1,4 @@
-.TH AHRIMAN "1" "2024\-09\-04" "ahriman" "Generated Python Manual"
+.TH AHRIMAN "1" "2024\-09\-19" "ahriman" "Generated Python Manual"
 .SH NAME
 ahriman
 .SH SYNOPSIS

src/ahriman/__init__.py

@@ -17,4 +17,4 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-__version__ = "2.14.1"
+__version__ = "2.14.2"

src/ahriman/core/alpm/repo.py

@@ -68,7 +68,7 @@ class Repo(LazyLogging):
             path(Path): path to archive to add
         """
         check_output(
-            "repo-add", *self.sign_args, "--remove", str(self.repo_path), str(path),
+            "repo-add", *self.sign_args, "-R", str(self.repo_path), str(path),
             exception=BuildError.from_process(path.name),
             cwd=self.paths.repository,
             logger=self.logger,
@@ -78,13 +78,8 @@ class Repo(LazyLogging):
         """
         create empty repository database. It just calls add with empty arguments
         """
-        # since pacman-6.1.0 repo-add doesn't create empty database in case if no packages supplied
-        # this code creates empty files instead
-        if self.repo_path.exists():
-            return  # database is already created, skip this part
-
-        self.repo_path.touch(exist_ok=True)
-        (self.paths.repository / f"{self.name}.db").symlink_to(self.repo_path)
+        check_output("repo-add", *self.sign_args, str(self.repo_path),
+                     cwd=self.paths.repository, logger=self.logger, user=self.uid)
 
     def remove(self, package: str, filename: Path) -> None:
         """

tests/ahriman/core/alpm/test_repo.py

@@ -26,28 +26,13 @@ def test_repo_add(repo: Repo, mocker: MockerFixture) -> None:
 
 def test_repo_init(repo: Repo, mocker: MockerFixture) -> None:
     """
-    must create empty database files
+    must call repo-add with empty package list on repo initializing
     """
-    mocker.patch("pathlib.Path.exists", return_value=False)
-    touch_mock = mocker.patch("pathlib.Path.touch")
-    symlink_mock = mocker.patch("pathlib.Path.symlink_to")
+    check_output_mock = mocker.patch("ahriman.core.alpm.repo.check_output")
 
     repo.init()
-    touch_mock.assert_called_once_with(exist_ok=True)
-    symlink_mock.assert_called_once_with(repo.repo_path)
-
-
-def test_repo_init_skip(repo: Repo, mocker: MockerFixture) -> None:
-    """
-    must do not create files if database already exists
-    """
-    mocker.patch("pathlib.Path.exists", return_value=True)
-    touch_mock = mocker.patch("pathlib.Path.touch")
-    symlink_mock = mocker.patch("pathlib.Path.symlink_to")
-
-    repo.init()
-    touch_mock.assert_not_called()
-    symlink_mock.assert_not_called()
+    check_output_mock.assert_called_once()  # it will be checked later
+    assert check_output_mock.call_args[0][0] == "repo-add"
 
 
 def test_repo_remove(repo: Repo, mocker: MockerFixture) -> None: