Skip to content

feat: clean up packages#135

Merged
NN708 merged 3 commits intoVanilla-OS:devfrom
vanilla-arm:dev
Jan 15, 2026
Merged

feat: clean up packages#135
NN708 merged 3 commits intoVanilla-OS:devfrom
vanilla-arm:dev

Conversation

@NN708
Copy link
Member

@NN708 NN708 commented Jan 14, 2026

This PR optimizes the packages by:

  1. Removing redundant packages that are already included as dependencies to simplify the recipe.
  2. Removing desktop-related packages, as these should be managed in desktop-image.
  3. Removing Apx, which can now be installed via Flatpak (Deliver Apx GUI as a Flatpak first-setup#406).

@NN708 NN708 requested a review from a team January 14, 2026 16:30
Copy link
Member

@taukakao taukakao left a comment

Choose a reason for hiding this comment

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

Why are you removing apx?

@NN708
Copy link
Member Author

NN708 commented Jan 15, 2026

Why are you removing apx?

Apx is already packaged in Apx GUI Flatpak, so I think we should remove the duplication here to avoid having multiple versions of Apx on the system. Users could run flatpak run --command=apx org.vanillaos.ApxGUI if they still prefer using the Apx CLI.

The Distrobox bundled with Apx should also be removed, since it's already installed in #116.

@taukakao
Copy link
Member

I think not everyone should be expected to install the apx gui. Also, flatpak is not that well suited for CLI programs.

Both apx and distrobox are quite small so removing them doesn't really improve anything.

Including two versions of distrobox was done to more tightly control the version that apx uses, though I didn't make that decision and don't really care.

@NN708
Copy link
Member Author

NN708 commented Jan 15, 2026

I think not everyone should be expected to install the apx gui. Also, flatpak is not that well suited for CLI programs.

Both apx and distrobox are quite small so removing them doesn't really improve anything.

The issue is not their size. The Apx in Flatpak won't have access to the configuration on the host system. Instead, it will store its own config at ~/.var/app/org.vanillaos.ApxGUI/data/apx. This could lead to different configurations between GUI and CLI, which would likely confuse users.

I think we should ask users to handle the transition, similar to how they are expected to manually install Showtime (Vanilla-OS/first-setup#397), Papers (Vanilla-OS/first-setup#404), and upcoming changes like Ptyxis (Vanilla-OS/desktop-image#279) and Resources (Vanilla-OS/desktop-image#286). However, we should consider how to better guide users through this process.

Given that this has become more complicated, I have removed that commit from this PR and will open a separate one instead.

@NN708 NN708 merged commit f798924 into Vanilla-OS:dev Jan 15, 2026
9 of 10 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.

3 participants