try to improve parser

.github/workflows/setup.sh

@@ -15,6 +15,8 @@ pacman -Sy --noconfirm devtools git pyalpm python-inflection python-passlib pyth
 pacman -Sy --noconfirm --asdeps base-devel python-build python-flit python-installer python-tox python-wheel
 # optional dependencies
 if [[ -z $MINIMAL_INSTALL ]]; then
+    # VCS support
+    pacman -Sy --noconfirm breezy darcs mercurial subversion
     # web server
     pacman -Sy --noconfirm python-aioauth-client python-aiohttp python-aiohttp-apispec-git python-aiohttp-cors python-aiohttp-jinja2 python-aiohttp-security python-aiohttp-session python-cryptography python-jinja
     # additional features

Dockerfile

@@ -31,6 +31,7 @@ RUN useradd -m -d "/home/build" -s "/usr/bin/nologin" build && \
     echo "build ALL=(ALL) NOPASSWD: ALL" > "/etc/sudoers.d/build"
 COPY "docker/install-aur-package.sh" "/usr/local/bin/install-aur-package"
 ## install package dependencies
+## darcs is not installed by reasons, because it requires a lot haskell packages which dramatically increase image size
 RUN pacman -Sy --noconfirm --asdeps \
         devtools \
         git \
@@ -49,7 +50,9 @@ RUN pacman -Sy --noconfirm --asdeps \
         python-wheel \
         && \
     pacman -Sy --noconfirm --asdeps \
+        breezy \
         git \
+        mercurial \
         python-aiohttp \
         python-boto3 \
         python-cerberus \
@@ -58,6 +61,7 @@ RUN pacman -Sy --noconfirm --asdeps \
         python-matplotlib \
         python-systemd \
         rsync \
+        subversion \
         && \
     runuser -u build -- install-aur-package \
         python-aioauth-client \

docs/faq/general.rst

@@ -265,7 +265,11 @@ TL;DR
 How to update VCS packages
 ^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-Normally the service handles VCS packages correctly. The version is updated in clean chroot, no additional actions are required.
+Normally the service handles VCS packages correctly, however it requires additional dependencies:
+
+.. code-block:: shell
+
+   pacman -S breezy darcs mercurial subversion
 
 How to review changes before build
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

package/archlinux/PKGBUILD

@@ -9,7 +9,10 @@ url="https://github.com/arcan1s/ahriman"
 license=('GPL3')
 depends=('devtools>=1:1.0.0' 'git' 'pyalpm' 'python-inflection' 'python-passlib' 'python-pyelftools' 'python-requests' 'python-srcinfo')
 makedepends=('python-build' 'python-flit' 'python-installer' 'python-wheel')
-optdepends=('python-aioauth-client: web server with OAuth2 authorization'
+optdepends=('breezy: -bzr packages support'
+            'darcs: -darcs packages support'
+            'mercurial: -hg packages support'
+            'python-aioauth-client: web server with OAuth2 authorization'
             'python-aiohttp: web server'
             'python-aiohttp-apispec>=3.0.0: web server'
             'python-aiohttp-cors: web server'
@@ -23,7 +26,8 @@ optdepends=('python-aioauth-client: web server with OAuth2 authorization'
             'python-requests-unixsocket2: client report to web server by unix socket'
             'python-jinja: html report generation'
             'python-systemd: journal support'
-            'rsync: sync by using rsync')
+            'rsync: sync by using rsync'
+            'subversion: -svn packages support')
 source=("https://github.com/arcan1s/ahriman/releases/download/$pkgver/$pkgname-$pkgver.tar.gz"
         'ahriman.sysusers'
         'ahriman.tmpfiles')

src/ahriman/core/build_tools/task.py

@@ -98,13 +98,12 @@ class Task(LazyLogging):
             if self.include_debug_packages or not package.name.startswith(debug_package_prefix)
         ]
 
-    def build(self, sources_dir: Path, *, dry_run: bool = False, **kwargs: str | None) -> list[Path]:
+    def build(self, sources_dir: Path, **kwargs: str | None) -> list[Path]:
         """
         run package build
 
         Args:
             sources_dir(Path): path to where sources are
-            dry_run(bool, optional): do not perform build itself (Default value = False)
             **kwargs(str | None): environment variables to be passed to build processes
 
         Returns:
@@ -114,8 +113,6 @@ class Task(LazyLogging):
         command.extend(self.archbuild_flags)
         command.extend(["--"] + self.makechrootpkg_flags)
         command.extend(["--"] + self.makepkg_flags)
-        if dry_run:
-            command.extend(["--nobuild"])
         self.logger.info("using %s for %s", command, self.package.base)
 
         environment: dict[str, str] = {
@@ -162,3 +159,25 @@ class Task(LazyLogging):
             patch.write(sources_dir / "PKGBUILD")
 
         return last_commit_sha
+
+    def setup(self, sources_dir: Path) -> None:
+        """
+        setup chroot environment without building package itself. This function, in particular, useful in case if it is
+        required to refresh pkgver to the actual value without package building
+
+        Args:
+            sources_dir(Path): path to where sources are
+        """
+        command = [self.build_command, "-r", str(self.paths.chroot)]
+        command.extend(self.archbuild_flags)
+        command.extend(["--"] + self.makechrootpkg_flags)
+        command.extend(["--"] + self.makepkg_flags + ["--nobuild"])
+        self.logger.info("using %s for %s", command, self.package.base)
+
+        check_output(
+            *command,
+            exception=BuildError.from_process(self.package.base),
+            cwd=sources_dir,
+            logger=self.logger,
+            user=self.uid,
+        )

src/ahriman/models/package.py

@@ -435,7 +435,7 @@ class Package(LazyLogging):
         try:
             # create fresh chroot environment, fetch sources and - automagically - update PKGBUILD
             task.init(paths.cache_for(self.base), [], None)
-            task.build(paths.cache_for(self.base), dry_run=False)
+            task.setup(paths.cache_for(self.base))
 
             pkgbuild = Pkgbuild.from_file(paths.cache_for(self.base) / "PKGBUILD")
 

src/ahriman/models/pkgbuild.py

@@ -184,7 +184,7 @@ class Pkgbuild(Mapping[str, str | list[str]]):
             raise ValueError("Function body wasn't found")
 
         # read the specified interval from source stream
-        io.seek(start_position - 1)  # start from the previous symbol ("{")
+        io.seek(start_position - 1)  # start from the previous symbol ({)
         content = io.read(end_position - start_position + 1)
 
         return content
@@ -202,7 +202,7 @@ class Pkgbuild(Mapping[str, str | list[str]]):
             tuple[str, PkgbuildPatch]: extracted a pair of key and its value
 
         Raises:
-            StopIteration: if iteration reaches the end of the file
+            StopIteration: if iteration reaches the end of the file'
         """
         # simple assignment rule
         if (match := Pkgbuild._STRING_ASSIGNMENT.match(token)) is not None:
@@ -223,14 +223,6 @@ class Pkgbuild(Mapping[str, str | list[str]]):
                 value = Pkgbuild._parse_function(parser)
                 return token, PkgbuildPatch(key, value)  # this is not mistake, assign to token without ()
 
-            # special function case, where "(" and ")" are separated tokens, e.g. "pkgver ( )"
-            case PkgbuildToken.ArrayStarts if Pkgbuild._FUNCTION_DECLARATION.match(token):
-                next_token = parser.get_token()
-                if next_token == PkgbuildToken.ArrayEnds:  # replace closing bracket with "()"
-                    next_token = PkgbuildToken.FunctionDeclaration
-                parser.push_token(next_token)  # type: ignore[arg-type]
-                return Pkgbuild._parse_token(token, parser)
-
             # some random token received without continuation, lets guess it is empty assignment (i.e. key=)
             case other if other is not None:
                 return Pkgbuild._parse_token(other, parser)