Skip to content

Packages (Linux): detects rum overlay packages - #2597

Merged
CarterLi merged 2 commits into
fastfetch-cli:devfrom
ddy314:feature/rum-overlay-packages
Sep 21, 2026
Merged

CarterLi merged 2 commits into
fastfetch-cli:devfrom
ddy314:feature/rum-overlay-packages

Conversation

@ddy314

@ddy314 ddy314 commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Count packages in RakuOS rum's overlay RPM database using the existing cached SQLite lookup. Keep the count separate from the existing rpm field.

Related issue

Closes #2511

Changes

  • Read /var/lib/rakuos/rum-rpmdb/rpmdb.sqlite, expose rum in default output, JSON, custom formats, and the disabled list.
  • Place the format argument and help entry after rpm in alphabetical order, and update the schema and changelog.

Validation

Built with GCC on Linux and passed all 16 CTest tests. Eight checks using an isolated target root and SQLite fixtures cover independent base/overlay counts, cache reuse and invalidation, disabled detection, named formats, default output, and missing/empty databases. The base commit reports only the three fixture RPM packages; the patched build reports three RPM plus five rum packages, totaling eight.

The normal host build also reports its existing pacman package count correctly. RakuOS itself was not booted for testing. Optional Vulkan and librpm support were disabled in the local build; SQLite support was enabled. The all-modules smoke run exceeded a 40-second limit, so only the Packages smoke and CTest results are claimed here.

Screenshots

No GUI changes. Fixture output: 3 (rpm), 5 (rum).

Checklist

  • I have tested my changes locally.

AI assistance: Codex helped implement and test this change and draft this description.

Comment thread src/modules/packages/packages.c Outdated
FF_ARG(hpkgAll, "hpkg-all"),
FF_ARG(nixAll, "nix-all"),
FF_ARG(counts.all, "all"),
FF_ARG(counts.rum, "rum"),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be in alphabet order, before all -all entries.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved rum immediately after rpm in both the format argument list and the help entries in 6a57e65. Rebuilt and reran all 16 CTest tests plus the eight isolated RPM/rum fixture checks; all passed.

Comment thread src/modules/packages/packages.c Outdated
{ "Total number of all hpkg packages", "hpkg-all" },
{ "Total number of all nix packages", "nix-all" },
{ "Number of all packages", "all" },
{ "Number of rum overlay packages", "rum" },

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

@CarterLi
CarterLi merged commit 203ccb3 into fastfetch-cli:dev Sep 21, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] support for listing rum package count

2 participants