.ci/ruff.sh

@@ -8,15 +8,19 @@ if [ "$(id -u)" = 0 ]; then
 	exec su "${TESTUSER:-build}" -c "sh -e $0"
 fi
 
+DID_FAIL=0
+
 set -x
 
 # __init__.py with additional ignore:
 # F401: imported, but not used
 # shellcheck disable=SC2046
-ruff check --ignore "F401" $(find . -not -path '*/venv/*' -name '__init__.py')
+ruff check --ignore "F401" $(find . -not -path '*/venv/*' -name '__init__.py') || DID_FAIL=1
 
 # Check all other files
-ruff check --exclude=__init__.py .
+ruff check --exclude=__init__.py . || DID_FAIL=1
 
 # Check formatting
-ruff format --diff
+ruff format --diff || DID_FAIL=1
+
+exit $DID_FAIL

docs/api/pmb.commands.rst

@@ -20,6 +20,14 @@ pmb.commands.base module
    :undoc-members:
    :show-inheritance:
 
+pmb.commands.flasher module
+---------------------------
+
+.. automodule:: pmb.commands.flasher
+   :members:
+   :undoc-members:
+   :show-inheritance:
+
 pmb.commands.index module
 -------------------------
 

pmb/chroot/test_apk.py

@@ -21,7 +21,7 @@ def apk_mocks(monkeypatch) -> dict | None:
 
     def _apkindex_package(
         _package: str, _arch: Arch, _must_exist: bool = False, indexes=None
-    ) -> ApkindexBlock:
+    ) -> ApkindexBlock | None:
         if _package == "package1":
             return ApkindexBlock(
                 arch=_arch,
@@ -68,6 +68,8 @@ def apk_mocks(monkeypatch) -> dict | None:
                 version="5.5-r0",
             )
 
+        return None
+
     monkeypatch.setattr(pmb.parse.apkindex, "package", _apkindex_package)
     return None
 

pmb/commands/__init__.py

@@ -11,6 +11,7 @@ from pmb.helpers import frontend
 
 from .base import Command
 from .aportgen import Aportgen
+from .flasher import Flasher
 from .log import Log
 from .index import Index
 from .repo_bootstrap import RepoBootstrap
@@ -31,7 +32,6 @@ unmigrated_commands = [
     "export",
     "sideload",
     "netboot",
-    "flasher",
     "initfs",
     "qemu",
     "aportupgrade",
@@ -65,6 +65,16 @@ def run_command(args: PmbArgs):
     match args.action:
         case "aportgen":
             command = Aportgen(args.packages, args.fork_alpine, args.fork_alpine_retain_branch)
+        case "flasher":
+            command = Flasher(
+                args.action_flasher,
+                args.autoinstall,
+                getattr(args, "cmdline", None),
+                args.flash_method,
+                getattr(args, "no_reboot", None),
+                getattr(args, "partition", None),
+                getattr(args, "resume", None),
+            )
         case "log":
             command = Log(args.clear_log, args.lines)
         case "index":

pmb/commands/flasher.py

+# Copyright 2023 Oliver Smith
+# Copyright 2024 Stefan Hansson
+# SPDX-License-Identifier: GPL-3.0-or-later
+
+import pmb.parse.deviceinfo
+from pmb import commands
+from pmb.core.context import get_context
+from pmb.flasher.frontend import flash_lk2nd, kernel, list_flavors, rootfs, sideload
+from pmb.helpers import logging
+
+
+class Flasher(commands.Command):
+    def __init__(
+        self,
+        action_flasher: str,
+        autoinstall: bool,
+        cmdline: str | None,
+        flash_method: str,
+        no_reboot: bool | None,
+        partition: str | None,
+        resume: bool | None,
+    ) -> None:
+        self.action_flasher = action_flasher
+        self.autoinstall = autoinstall
+        self.cmdline = cmdline
+        self.flash_method = flash_method
+        self.no_reboot = no_reboot
+        self.partition = partition
+        self.resume = resume
+
+    def run(self) -> None:
+        context = get_context()
+        action = self.action_flasher
+        device = context.config.device
+        deviceinfo = pmb.parse.deviceinfo()
+        method = self.flash_method or deviceinfo.flash_method
+
+        if method == "none" and action in ["boot", "flash_kernel", "flash_rootfs", "flash_lk2nd"]:
+            logging.info("This device doesn't support any flash method.")
+            return
+
+        if action in ["boot", "flash_kernel"]:
+            kernel(deviceinfo, method, action == "boot", self.autoinstall)
+        elif action == "flash_rootfs":
+            rootfs(deviceinfo, method)
+        elif action == "flash_vbmeta":
+            logging.info("(native) flash vbmeta.img with verity disabled flag")
+            pmb.flasher.run(
+                deviceinfo,
+                method,
+                "flash_vbmeta",
+                cmdline=self.cmdline,
+                no_reboot=self.no_reboot,
+                partition=self.partition,
+                resume=self.resume,
+            )
+        elif action == "flash_dtbo":
+            logging.info("(native) flash dtbo image")
+            pmb.flasher.run(
+                deviceinfo,
+                method,
+                "flash_dtbo",
+                cmdline=self.cmdline,
+                no_reboot=self.no_reboot,
+                partition=self.partition,
+                resume=self.resume,
+            )
+        elif action == "flash_lk2nd":
+            flash_lk2nd(deviceinfo, method)
+        elif action == "list_flavors":
+            list_flavors(device)
+        elif action == "list_devices":
+            pmb.flasher.run(
+                deviceinfo,
+                method,
+                "list_devices",
+                cmdline=self.cmdline,
+                no_reboot=self.no_reboot,
+                partition=self.partition,
+                resume=self.resume,
+            )
+        elif action == "sideload":
+            sideload(deviceinfo, method)

pmb/config/__init__.py

@@ -211,7 +211,11 @@ chroot_outdated = 3600 * 24 * 2
 # for the first time
 # IMPORTANT: the order here matters, it is the order these packages will
 # be built in (if needed). abuild must be first!
-build_packages = ["abuild", "apk-tools", "build-base", "ccache", "git"]
+#
+# NOTE: full hexdump is installed to workaround a bug in busybox,
+# see https://gitlab.postmarketos.org/postmarketOS/pmaports/-/issues/3268. This can be
+# reverted when the bug is properly fixed.
+build_packages = ["abuild", "apk-tools", "build-base", "ccache", "git", "hexdump"]
 
 #
 # PARSE

pmb/core/crosstool.py

-# Copyright 2024 Caleb Connolly
-# SPDX-License-Identifier: GPL-3.0-or-later
-
-import enum
-from pathlib import Path
-
-from pmb.core.chroot import ChrootType
-from pmb.types import PathString
-
-
-class CrossToolTarget(enum.Enum):
-    BUILDROOT = 0
-    ROOTFS = 1
-
-
-class CrossTool:
-    __target: CrossToolTarget
-    __package: str
-    __paths: list[Path]
-
-    def __init__(self, target: CrossToolTarget, package: str, paths: list[PathString]):
-        self.__target = target
-        self.__package = package
-        self.__paths = list(map(lambda p: Path(p) if isinstance(p, str) else p, paths))
-
-    def __repr__(self) -> str:
-        return f"CrossTool({self.__target}, {self.__package}, {self.__paths})"
-
-    @property
-    def package(self) -> str:
-        return self.__package
-
-    @property
-    def paths(self) -> list[Path]:
-        return self.__paths
-
-    def should_install(self, target: ChrootType) -> bool:
-        if target == ChrootType.BUILDROOT and self.__target == CrossToolTarget.BUILDROOT:
-            return True
-        if (
-            target == ChrootType.ROOTFS
-            or target == ChrootType.INSTALLER
-            and self.__target == CrossToolTarget.ROOTFS
-        ):
-            return True
-
-        return False

pmb/flasher/__init__.py

@@ -5,4 +5,3 @@ from pmb.flasher.init import install_depends
 from pmb.flasher.run import run
 from pmb.flasher.run import check_partition_blacklist
 from pmb.flasher.variables import variables
-from pmb.flasher.frontend import frontend

pmb/flasher/frontend.py

@@ -7,7 +7,6 @@ from pmb.helpers import logging
 
 import pmb.config
 from pmb.parse.deviceinfo import Deviceinfo
-from pmb.types import PmbArgs
 import pmb.flasher
 import pmb.install
 import pmb.chroot.apk
@@ -146,34 +145,3 @@ def flash_lk2nd(deviceinfo: Deviceinfo, method: str) -> None:
 
     logging.info("(native) flash lk2nd image")
     pmb.flasher.run(deviceinfo, method, "flash_lk2nd")
-
-
-def frontend(args: PmbArgs) -> None:
-    context = get_context()
-    action = args.action_flasher
-    device = context.config.device
-    deviceinfo = pmb.parse.deviceinfo()
-    method = args.flash_method or deviceinfo.flash_method
-
-    if method == "none" and action in ["boot", "flash_kernel", "flash_rootfs", "flash_lk2nd"]:
-        logging.info("This device doesn't support any flash method.")
-        return
-
-    if action in ["boot", "flash_kernel"]:
-        kernel(deviceinfo, method, action == "boot", args.autoinstall)
-    elif action == "flash_rootfs":
-        rootfs(deviceinfo, method)
-    elif action == "flash_vbmeta":
-        logging.info("(native) flash vbmeta.img with verity disabled flag")
-        pmb.flasher.run(deviceinfo, method, "flash_vbmeta")
-    elif action == "flash_dtbo":
-        logging.info("(native) flash dtbo image")
-        pmb.flasher.run(deviceinfo, method, "flash_dtbo")
-    elif action == "flash_lk2nd":
-        flash_lk2nd(deviceinfo, method)
-    elif action == "list_flavors":
-        list_flavors(device)
-    elif action == "list_devices":
-        pmb.flasher.run(deviceinfo, method, "list_devices")
-    elif action == "sideload":
-        sideload(deviceinfo, method)

pmb/flasher/run.py

@@ -24,7 +24,16 @@ def check_partition_blacklist(deviceinfo: Deviceinfo, key, value):
         )
 
 
-def run(deviceinfo: Deviceinfo, method: str, action: str, flavor=None):
+def run(
+    deviceinfo: Deviceinfo,
+    method: str,
+    action: str,
+    flavor: str | None = None,
+    cmdline: str | None = None,
+    no_reboot: bool | None = None,
+    partition: str | None = None,
+    resume: bool | None = None,
+) -> None:
     pmb.flasher.init(deviceinfo.codename, method)
 
     # Verify action
@@ -42,9 +51,7 @@ def run(deviceinfo: Deviceinfo, method: str, action: str, flavor=None):
         )
 
     # Variable setup
-    # FIXME: handle argparsing and pass in only the args we need.
-    args = pmb.helpers.args.please_i_really_need_args()
-    fvars = pmb.flasher.variables(args, flavor, method)
+    fvars = pmb.flasher.variables(flavor, method, cmdline, no_reboot, partition, resume)
 
     # vbmeta flasher requires vbmeta partition to be explicitly specified
     if action == "flash_vbmeta" and not fvars["$PARTITION_VBMETA"]:
@@ -69,12 +76,12 @@ def run(deviceinfo: Deviceinfo, method: str, action: str, flavor=None):
             "Deviceinfo_reference>"
         )
 
-    if args.no_reboot and ("flash" not in action or method != "heimdall-bootimg"):
+    if no_reboot and ("flash" not in action or method != "heimdall-bootimg"):
         raise RuntimeError(
             "The '--no-reboot' option is only" " supported when flashing with heimall-bootimg."
         )
 
-    if args.resume and ("flash" not in action or method != "heimdall-bootimg"):
+    if resume and ("flash" not in action or method != "heimdall-bootimg"):
         raise RuntimeError(
             "The '--resume' option is only" " supported when flashing with heimall-bootimg."
         )

pmb/flasher/variables.py

@@ -3,15 +3,21 @@
 import pmb.config.pmaports
 from pmb.core.chroot import Chroot
 from pmb.core.context import get_context
-from pmb.types import PmbArgs
 
 
-def variables(args: PmbArgs, flavor: str, method: str):
+def variables(
+    flavor: str | None,
+    method: str,
+    cmdline: str | None,
+    no_reboot: bool | None,
+    partition: str | None,
+    resume: bool | None,
+) -> dict[str, str]:
     device = get_context().config.device
     deviceinfo = pmb.parse.deviceinfo()
     _cmdline = deviceinfo.kernel_cmdline or ""
-    if "cmdline" in args and args.cmdline:
-        _cmdline = args.cmdline
+    if cmdline:
+        _cmdline = cmdline
 
     flash_pagesize = deviceinfo.flash_pagesize
 
@@ -54,22 +60,22 @@ def variables(args: PmbArgs, flavor: str, method: str):
         _partition_vbmeta = deviceinfo.flash_heimdall_partition_vbmeta or None
         _partition_dtbo = deviceinfo.flash_heimdall_partition_dtbo or None
 
-    if "partition" in args and args.partition:
+    if partition:
         # Only one operation is done at same time so it doesn't matter
         # sharing the arg
-        _partition_kernel = args.partition
-        _partition_rootfs = args.partition
-        _partition_vbmeta = args.partition
-        _partition_dtbo = args.partition
+        _partition_kernel = partition
+        _partition_rootfs = partition
+        _partition_vbmeta = partition
+        _partition_dtbo = partition
 
     _dtb = deviceinfo.dtb + ".dtb"
 
     _no_reboot = ""
-    if getattr(args, "no_reboot", False):
+    if no_reboot:
         _no_reboot = "--no-reboot"
 
     _resume = ""
-    if getattr(args, "resume", False):
+    if resume:
         _resume = "--resume"
 
     fvars = {

pmb/helpers/frontend.py

@@ -403,10 +403,6 @@ def install(args: PmbArgs) -> None:
     pmb.install.install(args)
 
 
-def flasher(args: PmbArgs) -> None:
-    pmb.flasher.frontend(args)
-
-
 def export(args: PmbArgs) -> None:
     pmb.export.frontend(args)
 

pmb/helpers/other.py

@@ -82,10 +82,16 @@ def migrate_work_folder():
     # Read current version
     context = get_context()
     current = 0
+    suffix: str | None = None
     path = context.config.work / "version"
     if os.path.exists(path):
         with open(path) as f:
-            current = int(f.read().rstrip())
+            # pmb 2.3.x added a suffix due to conflicting work versions
+            # We need to be able to handle that going forward
+            version_parts = f.read().rstrip().split("-")
+            current = int(version_parts[0])
+            if len(version_parts) == 2:
+                suffix = version_parts[1]
 
     # Compare version, print warning or do nothing
     required = pmb.config.work_version
@@ -96,7 +102,8 @@ def migrate_work_folder():
         " (from version " + str(current) + " to " + str(required) + ")!"
     )
 
-    if current == 6:
+    # version 6 and version 7 from 2.3.x branch are equivalent for this and we need to migrate
+    if current == 6 or (current == 7 and suffix == "2.x"):
         # Ask for confirmation
         logging.info("Changelog:")
         logging.info("* Major refactor for pmb 3.0.0")

pmb/install/recovery.py

@@ -21,7 +21,14 @@ def create_zip(args: PmbArgs, chroot: Chroot, device: str):
     flavor = pmb.helpers.frontend._parse_flavor(device)
     deviceinfo = pmb.parse.deviceinfo()
     method = deviceinfo.flash_method
-    fvars = pmb.flasher.variables(args, flavor, method)
+    fvars = pmb.flasher.variables(
+        flavor,
+        method,
+        getattr(args, "cmdline", None),
+        getattr(args, "no_reboot", None),
+        getattr(args, "partition", None),
+        getattr(args, "resume", None),
+    )
 
     # Install recovery installer package in buildroot
     pmb.chroot.apk.install(["postmarketos-android-recovery-installer"], chroot)

pmb/parse/apkindex.py

@@ -91,13 +91,23 @@ def parse_next_block(path: Path, lines: list[str]) -> ApkindexBlock | None:
                 ret[key].append(value)
         else:
             ret[key] = []
+    provider_priority = ret.get("provider_priority")
+    if provider_priority:
+        if not provider_priority.isdigit():
+            raise RuntimeError(
+                f"Invalid provider_priority: '{provider_priority}' parsing block {ret}"
+            )
+        provider_priority = int(provider_priority)
+    else:
+        provider_priority = None
+
     return ApkindexBlock(
         arch=Arch.from_str(ret["arch"]),
         depends=ret["depends"],
         origin=ret.get("origin"),
         pkgname=ret["pkgname"],
         provides=ret["provides"],
-        provider_priority=ret.get("provider_priority"),
+        provider_priority=provider_priority,
         timestamp=ret.get("timestamp"),
         version=ret["version"],
     )
@@ -306,7 +316,7 @@ def parse_blocks(path: Path) -> list[ApkindexBlock]:
 
 
 def cache_key(path: Path) -> str:
-    return str(path.relative_to(get_context().config.work))
+    return str(path.relative_to(get_context().config.work, walk_up=True))
 
 
 def clear_cache(path: Path) -> bool:

pmb/parse/test_apkindex.py

+from pmb.core.arch import Arch
+import pytest
+
+from .apkindex import parse as parse_apkindex
+
+example_apkindex = """
+C:Q1p+nGf5oBAmbU9FQvV4MhfEmWqVE=
+P:postmarketos-base-ui-networkmanager
+V:29-r1
+A:aarch64
+S:4185
+I:5376
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:busctl dnsmasq-dnssec-dbus>=2.90-r1 gojq networkmanager networkmanager-cli networkmanager-tui networkmanager-wifi networkmanager-wwan networkmanager-dnsmasq
+
+C:Q1j8i5C7tOHbhrd5OTtot1u1ZtteU=
+P:postmarketos-base-ui-networkmanager-usb-tethering
+V:29-r1
+A:aarch64
+S:3986
+I:5206
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber
+i:postmarketos-base-ui-networkmanager=29-r1
+
+C:Q10FDERw8SRwuk6ITfWNyJ4GbVE2I=
+P:postmarketos-base-ui-openrc
+V:29-r1
+A:aarch64
+S:1570
+I:1
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony-openrc dbus-openrc haveged-openrc util-linux-openrc /bin/sh
+i:postmarketos-base-ui=29-r1 openrc
+
+C:Q1inMHA1+gsS8U50j8odasUNy5TVw=
+P:postmarketos-base-ui-openrc-settingsd
+V:29-r1
+A:aarch64
+S:1824
+I:103
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber /bin/sh
+i:postmarketos-base-ui=29-r1 openrc-settingsd-openrc
+
+C:Q1iB2PiTO0w29T4kRUSjqLOyKcCwY=
+P:postmarketos-base-ui-qt-tweaks
+V:29-r1
+A:aarch64
+S:1661
+I:47
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber
+
+C:Q1aQ5UEOC902h2atx0fqDhahpkDY8=
+P:postmarketos-base-ui-qt-wayland
+V:29-r1
+A:aarch64
+S:1687
+I:86
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber
+
+C:Q16q7k1Z5HAJe5qqeXfTIn62QQoBs=
+P:postmarketos-base-ui-tinydm
+V:29-r1
+A:aarch64
+S:2264
+I:571
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber
+p:postmarketos-base-tinydm=29-r1
+i:postmarketos-base-ui=29-r1 tinydm-openrc
+
+C:Q11IrmIOlqZX5BzKhLsce+U+S+N3I=
+P:postmarketos-base-ui-wayland
+V:29-r1
+A:aarch64
+S:1361
+I:0
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:chrony dbus haveged nftables openssh-server-pam pipewire postmarketos-artwork-icons postmarketos-base shadow tzdata util-linux wireless-regdb wireplumber
+i:postmarketos-base-ui=29-r1 wayland-server-libs
+
+C:Q1bDAQG9F9mi7Mi5CT3Csiq4QU52w=
+P:postmarketos-base-ui-wifi-iwd
+V:29-r1
+A:aarch64
+S:1650
+I:26
+T:Use iwd as the WiFi backend (but may not work with all devices)
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+k:90
+D:iwd
+p:postmarketos-base-ui-wifi=29-r1
+
+C:Q1s7shLQ3sKNxKABRyQRBQ11RwkuU=
+P:postmarketos-base-ui-wifi-iwd-openrc
+V:29-r1
+A:aarch64
+S:1405
+I:1
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:iwd-openrc openrc /bin/sh
+i:postmarketos-base-ui-wifi-iwd=29-r1 openrc
+
+C:Q1C23f0J5PBrNAuZiAhzCJhVpNe5c=
+P:postmarketos-base-ui-wifi-wpa_supplicant
+V:29-r1
+A:aarch64
+S:1262
+I:0
+T:Use wpa_supplicant as the WiFi backend.
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+k:100
+D:wpa_supplicant
+p:postmarketos-base-ui-wifi=29-r1
+
+C:Q1LfWeOxxgIM3UE/QQBczMpJw17hY=
+P:postmarketos-base-ui-wifi-wpa_supplicant-openrc
+V:29-r1
+A:aarch64
+S:1691
+I:37
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:wpa_supplicant-openrc openrc /bin/sh
+i:postmarketos-base-ui-wifi-wpa_supplicant=29-r1 openrc
+
+C:Q1yB3CVUFMOjnLOOEAUIUUpJJV8g0=
+P:postmarketos-base-ui-x11
+V:29-r1
+A:aarch64
+S:1587
+I:22
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:libinput xf86-input-libinput xf86-video-fbdev
+p:postmarketos-base-x11=29-r1
+i:postmarketos-base-ui=29-r1 xorg-server
+
+C:Q1TgE1jgGt1PUb/Zwbi6rDRt3b8go=
+P:postmarketos-initramfs
+V:3.3.5-r2
+A:aarch64
+S:16530
+I:143360
+T:Base files for the postmarketOS initramfs / initramfs-extra
+U:https://postmarketos.org
+L:GPL-2.0-or-later
+o:postmarketos-initramfs
+m:Caleb Connolly <caleb@postmarketos.org>
+t:1728049308
+c:f3a4285732781d5577bad06046b7bbeaf132ce1f-dirty
+k:10
+D:blkid btrfs-progs buffyboard busybox-extras bzip2 cryptsetup device-mapper devicepkg-utils>=0.2.0 dosfstools e2fsprogs e2fsprogs-extra f2fs-tools font-terminus iskey kmod libinput-libs lz4 multipath-tools parted postmarketos-fde-unlocker postmarketos-mkinitfs>=2.2 udev unudhcpd util-linux-misc xz
+p:postmarketos-ramdisk=3.3.5-r2"""
+
+
+def test_apkindex_parse(tmp_path) -> None:
+    tmpfile = tmp_path / "APKINDEX.1"
+    print(tmp_path)
+    f = open(tmpfile, "w")
+    f.write(example_apkindex)
+    f.close()
+
+    blocks = parse_apkindex(tmpfile, True)
+    for k, v in blocks.items():
+        print(f"{k}: {v}")
+
+    # Even though there's only 14 entries, there are some virtual packages
+    assert len(blocks) == 18
+
+    # Check that the postmarketos-ramdisk virtual package is handled correctly
+    # and that it's one provider (postmarketos-initramfs) is declared
+    assert "postmarketos-ramdisk" in blocks.keys()
+    assert "postmarketos-initramfs" in blocks["postmarketos-ramdisk"].keys()
+    assert (
+        blocks["postmarketos-ramdisk"]["postmarketos-initramfs"]
+        == blocks["postmarketos-initramfs"]["postmarketos-initramfs"]
+    )
+
+    initramfs = blocks["postmarketos-initramfs"]["postmarketos-initramfs"]
+    assert initramfs.pkgname == "postmarketos-initramfs"
+    assert initramfs.provides == ["postmarketos-ramdisk"]
+    assert initramfs.provider_priority == 10
+    assert initramfs.depends == [
+        "blkid",
+        "btrfs-progs",
+        "buffyboard",
+        "busybox-extras",
+        "bzip2",
+        "cryptsetup",
+        "device-mapper",
+        "devicepkg-utils",
+        "dosfstools",
+        "e2fsprogs",
+        "e2fsprogs-extra",
+        "f2fs-tools",
+        "font-terminus",
+        "iskey",
+        "kmod",
+        "libinput-libs",
+        "lz4",
+        "multipath-tools",
+        "parted",
+        "postmarketos-fde-unlocker",
+        "postmarketos-mkinitfs",
+        "udev",
+        "unudhcpd",
+        "util-linux-misc",
+        "xz",
+    ]
+
+    tinydm = blocks["postmarketos-base-ui-tinydm"]["postmarketos-base-ui-tinydm"]
+    # Without the version!
+    assert tinydm.provides == ["postmarketos-base-tinydm"]
+    assert tinydm.version == "29-r1"
+    assert tinydm.arch == Arch.aarch64
+
+    wayland = blocks["postmarketos-base-ui-wayland"]["postmarketos-base-ui-wayland"]
+    # Doesn't provide an explicit version
+    assert wayland.provides == []
+    assert wayland.origin == "postmarketos-base-ui"
+
+    networkmanager = blocks["postmarketos-base-ui-networkmanager"][
+        "postmarketos-base-ui-networkmanager"
+    ]
+    assert networkmanager.provider_priority is None
+
+
+def test_apkindex_parse_bad_priority(tmp_path) -> None:
+    tmpfile = tmp_path / "APKINDEX.2"
+    f = open(tmpfile, "w")
+    # A snippet of the above example but with the provider_priority
+    # of postmarketos-initramfs set to a non-integer value
+    f.write(
+        """
+C:Q1yB3CVUFMOjnLOOEAUIUUpJJV8g0=
+P:postmarketos-base-ui-x11
+V:29-r1
+A:aarch64
+S:1587
+I:22
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+o:postmarketos-base-ui
+m:Clayton Craft <clayton@craftyguy.net>
+t:1729538699
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:libinput xf86-input-libinput xf86-video-fbdev
+p:postmarketos-base-x11=29-r1
+i:postmarketos-base-ui=29-r1 xorg-server
+
+C:Q1TgE1jgGt1PUb/Zwbi6rDRt3b8go=
+P:postmarketos-initramfs
+V:3.3.5-r2
+A:aarch64
+S:16530
+I:143360
+T:Base files for the postmarketOS initramfs / initramfs-extra
+U:https://postmarketos.org
+L:GPL-2.0-or-later
+o:postmarketos-initramfs
+m:Caleb Connolly <caleb@postmarketos.org>
+t:1728049308
+c:f3a4285732781d5577bad06046b7bbeaf132ce1f-dirty
+k:beep
+D:blkid btrfs-progs buffyboard busybox-extras bzip2 cryptsetup device-mapper devicepkg-utils>=0.2.0 dosfstools e2fsprogs e2fsprogs-extra f2fs-tools font-terminus iskey kmod libinput-libs lz4 multipath-tools parted postmarketos-fde-unlocker postmarketos-mkinitfs>=2.2 udev unudhcpd util-linux-misc xz
+p:postmarketos-ramdisk=3.3.5-r2"""
+    )
+    f.close()
+
+    # We expect a RuntimeError to be raised when provider_priority is not
+    # an integer
+    with pytest.raises(RuntimeError):
+        parse_apkindex(tmpfile, True)
+
+
+def test_apkindex_parse_missing_optionals(tmp_path) -> None:
+    tmpfile = tmp_path / "APKINDEX.3"
+    f = open(tmpfile, "w")
+    # A snippet of the above example but with a missing timestamp
+    # and origin fields
+    f.write(
+        """
+C:Q1yB3CVUFMOjnLOOEAUIUUpJJV8g0=
+P:postmarketos-base-ui-x11
+V:29-r1
+A:aarch64
+S:1587
+I:22
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+m:Clayton Craft <clayton@craftyguy.net>
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:libinput xf86-input-libinput xf86-video-fbdev
+p:postmarketos-base-x11=29-r1
+i:postmarketos-base-ui=29-r1 xorg-server"""
+    )
+    f.close()
+
+    # We expect parsing to succeed when the timestamp is missing
+    parse_apkindex(tmpfile, True)
+
+
+def test_apkindex_parse_trailing_newline(tmp_path) -> None:
+    tmpfile = tmp_path / "APKINDEX.4"
+    f = open(tmpfile, "w")
+    # A snippet of the above example but with additional
+    # trailing newlines
+    f.write(
+        """
+C:Q1yB3CVUFMOjnLOOEAUIUUpJJV8g0=
+P:postmarketos-base-ui-x11
+V:29-r1
+A:aarch64
+S:1587
+I:22
+T:Meta package for minimal postmarketOS UI base
+U:https://postmarketos.org
+L:GPL-3.0-or-later
+m:Clayton Craft <clayton@craftyguy.net>
+c:901cb9520450a1e88ded95ac774e83f6b2cfbba3-dirty
+D:libinput xf86-input-libinput xf86-video-fbdev
+p:postmarketos-base-x11=29-r1
+i:postmarketos-base-ui=29-r1 xorg-server
+
+
+"""
+    )
+    f.close()
+
+    # We expect parsing to succeed when the timestamp is missing
+    parse_apkindex(tmpfile, True)