[System Updates Manager] Add adapters for pipx, xbps, and gearlever - #488
Open
YoshiWalsh wants to merge 9 commits into
Open
[System Updates Manager] Add adapters for pipx, xbps, and gearlever#488YoshiWalsh wants to merge 9 commits into
YoshiWalsh wants to merge 9 commits into
Conversation
(XBPS is the package manager for Void Linux, and can also be used on other distros)
Gear Lever (mijorus/gearlever) is a tool for managing AppImages. Note that the closest equivalent Gear Lever has to an "appid" is the full path to the .appimage file. Gear Lever can manage appimages in many different directories, so this cannot be omitted. Currently System Updates Manager appids are not allowed to contain / characters, so these are converted to : characters as a workaround. A better solution would be to allow / characters in appids. There should be no injection risk here as long as quotation marks are correctly used in the `update_package_command`.
Also bumps plugin version
Contributor
|
CC @tordex |
YoshiWalsh
marked this pull request as draft
August 26, 2026 14:48
Author
|
Converted back to draft because I think I need to do further testing on the XBPS adapter. |
Contributor
|
Awesome! New adapters are welcome! One note about xbps adapter. You should use |
Author
|
That makes sense! When I tested it, somehow it worked with sudo... once. I think I may have run some other command with sudo before launching noctalia, and I tested it during the sudo timeout. I converted this back to draft when I realised I couldn't get the XBPS stuff to work a second time. When I'm back at my PC I'll switch to using pkexec and test it again. |
YoshiWalsh
marked this pull request as ready for review
August 30, 2026 06:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Plugin
tordex/system-updaterplugin.toml)What it does
This PR adds three new adapters:
External dependencies
jq,grep,sed,andcoreutils.Testing
I have tried checking for updates, installing an update to one package, and installing all updates for each of the three adapters.
Screenshots / Videos
No significant changes
Checklist
Ready-for-review requirement: Every box in this section must be checked. If any statement is not true, keep the
pull request as Draft. An explanation does not replace a required check.
idafter the/inplugin.tomlexactly.plugin.toml,README.md,thumbnail.webp, andtranslations/en.json.README.mdfollows theREADME template, documents
every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.
thumbnail.webpwith the thumbnail generator.versionfollows semver and is bumped in this PR;plugin_apiis the oldest API level this plugin requires.understand that language well enough to review and maintain it (no unreviewed machine/LLM translations).
catalog.toml; CI generates it.Code review attestation
Plugins run as trusted, unsandboxed Luau in the user's session. Confirm:
Ready-for-review requirement: Every attestation below must be checked.
licensedeclared inplugin.toml.