Skip to content

fix(packaging): name the launcher icon dev.beyondhumane.Arca - #56

Merged
devin-ai-integration[bot] merged 1 commit into
mainfrom
devin/1792245939-icon-name
Oct 17, 2026
Merged

devin-ai-integration[bot] merged 1 commit into
mainfrom
devin/1792245939-icon-name

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Launchers resolved Icon=arca to the active icon theme first, and themes such as Colloid ship an unrelated arca.svg, so Arca showed another app's icon instead of its own (hicolor is only the fallback). The icon is now installed and requested under a reverse-DNS name nobody else can shadow:

Icon=arca  ->  Icon=dev.beyondhumane.Arca
share/icons/hicolor/256x256/apps/arca.png  ->  .../dev.beyondhumane.Arca.png
AppDir/arca.png + .DirIcon                 ->  AppDir/dev.beyondhumane.Arca.png + .DirIcon

Applied in all four places that name the file: packaging/linux/arca.desktop, packaging/linux/build.sh (.deb staging and AppImage), packaging/linux/nfpm.yaml, and makeDesktopItem { icon } + postInstall in packaging/nix/package.nix.

Deliberately unchanged: the desktop file stays arca.desktop and the window keeps with_app_id("arca"), so compositors and mimeapps.list entries keep associating the window with its desktop entry; only the icon name moves.

Link to Devin session: https://app.devin.ai/sessions/19c0b631a68145159a99f790aaa6eedd
Open in Devin Desktop: https://app.devin.ai/desktop/session/19c0b631a68145159a99f790aaa6eedd?variant=devin
Requested by: @Villoh

Icon themes such as Colloid ship an unrelated 'arca' icon, and the
active theme wins over hicolor, so the launcher showed the wrong one.
The desktop file, the .deb, the AppImage and the Nix package now all use
a reverse-DNS icon name nobody else can shadow. The desktop file name
and the app_id stay 'arca'.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review October 17, 2026 18:16
@devin-ai-integration
devin-ai-integration Bot merged commit e8a0597 into main Oct 17, 2026
5 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.

1 participant