Skip to content

refactor: specialized impl for getting apps installed via flatpak#6

Merged
SteveLauC merged 2 commits intomainfrom
fix/revert_linux_traversal_depth
May 12, 2025
Merged

refactor: specialized impl for getting apps installed via flatpak#6
SteveLauC merged 2 commits intomainfrom
fix/revert_linux_traversal_depth

Conversation

@SteveLauC
Copy link
Copy Markdown
Member

@SteveLauC SteveLauC commented May 12, 2025

Add a specialized implementation for retrieving apps installed via Flatpak

This change:

  1. Fixes a bug where the same app could be reported twice

    Every app installed via flatpak has 2 desktop files under its directory, take com.google.Chrome as an example:

$ find . -name "com.google.Chrome.desktop"
./x86_64/stable/ca5617c1195bd48bd3cfae549ea738f5bfc51f78aec1e7b025948339140b6abf/export/share/applications/com.google.Chrome.desktop
./x86_64/stable/ca5617c1195bd48bd3cfae549ea738f5bfc51f78aec1e7b025948339140b6abf/files/share/applications/com.google.Chrome.desktop

So our generic implementation will treat these 2 desktop files as 2 apps.

  1. Improves the performance of the get_all_apps() interface

@SteveLauC SteveLauC merged commit 9a73821 into main May 12, 2025
3 checks passed
@SteveLauC SteveLauC deleted the fix/revert_linux_traversal_depth branch May 12, 2025 09:33
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.

1 participant