Skip to content

bsp: ship Armbian distributor logo (svg + wordmark png variants)#9674

Merged
igorpecovnik merged 2 commits intomainfrom
bsp-armbian-logo
Apr 14, 2026
Merged

bsp: ship Armbian distributor logo (svg + wordmark png variants)#9674
igorpecovnik merged 2 commits intomainfrom
bsp-armbian-logo

Conversation

@igorpecovnik
Copy link
Copy Markdown
Member

@igorpecovnik igorpecovnik commented Apr 14, 2026

Summary

`artifact-armbian-base-files.sh:169` writes `LOGO="armbian-logo"` into `/etc/os-release`, but no actual `armbian-logo` file ever lands on the rootfs. So GNOME Settings → About, KDE Info Center, and every other panel that reads `LOGO=` from os-release looks up `armbian-logo` in the icon search paths, finds nothing, and silently falls back to the upstream distro logo. Branding has been broken at the file-presence level for as long as that field has been there.

Fix

Ship three files via the BSP overlay tree, matching upstream Ubuntu's three-file layout in base-files:

file path size role
logomark `/usr/share/pixmaps/armbian-logo.svg` 30 KB what `LOGO=armbian-logo` resolves to (icon-spec lookup picks SVG first)
wordmark light `/usr/share/pixmaps/armbian-logo-text.png` 8 KB, 400×64 for apps that prefer the text-bearing icon on a light background
wordmark dark `/usr/share/pixmaps/armbian-logo-text-dark.png` 8 KB, 400×64 same, dark background

Compare with upstream:
```
ubuntu-logo.svg (890 bytes)
ubuntu-logo-text.png (260×91, 5 KB)
ubuntu-logo-text-dark.png (256×89, 4.6 KB)
```

All three land on every Armbian image (CLI or desktop) via `armbian-bsp-cli`'s existing `rsync` of `packages/bsp/common/` (`armbian-bsp-cli-deb.sh:127`). No build-flow changes, no postinst hooks needed — they're picked up by the icon cache automatically as part of `/usr/share/pixmaps/`.

Follow-ups (separate PRs)

  • Once armbian-base-files migration (Phase 0) lands — move logo ownership from `armbian-bsp-cli` to the repacked `base-files` package for symmetry with upstream
  • Further SVG optimization: 30 KB → ~1 KB territory with more aggressive svgo passes; adequate as-is for the icon-cache use case

Test plan

  • Build a fresh image and confirm all three files are present under `/usr/share/pixmaps/`
  • On a desktop image: GNOME Settings → About displays the Armbian logo (not the Ubuntu/Debian fallback)
  • On KDE: System Information / Info Center displays the same
  • Boot/login screens that use the wordmark variant (depending on greeter) pick up `armbian-logo-text.png` / `-dark.png` instead of the upstream wordmark

artifact-armbian-base-files.sh writes LOGO=\"armbian-logo\" into
/etc/os-release, but no actual armbian-logo file ever lands on
the rootfs — so GNOME Settings -> About / KDE Info Center / etc.
look up `armbian-logo` in the icon search paths, find nothing,
and silently fall back to the upstream distro logo.

Ship the SVG via the BSP overlay tree at the canonical pixmaps
path (matches Ubuntu's `/usr/share/pixmaps/ubuntu-logo.svg` —
referenced by `LOGO=ubuntu-logo` in upstream's os-release).
Lands on every Armbian image, CLI or desktop, via armbian-bsp-cli's
existing rsync of packages/bsp/common/.

Follow-ups (separate work):
- armbian-logo-text.png + armbian-logo-text-dark.png wordmark
  variants matching upstream's two PNG companions
- once armbian-base-files migration (Phase 0) lands, move logo
  ownership from BSP to base-files for symmetry with upstream
@igorpecovnik igorpecovnik requested a review from a team as a code owner April 14, 2026 22:29
@igorpecovnik igorpecovnik requested review from lanefu and sgjava and removed request for a team April 14, 2026 22:29
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 14, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • packages/bsp/common/usr/share/pixmaps/armbian-logo-text-dark.png is excluded by !**/*.png
  • packages/bsp/common/usr/share/pixmaps/armbian-logo-text.png is excluded by !**/*.png
  • packages/bsp/common/usr/share/pixmaps/armbian-logo.svg is excluded by !**/*.svg

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 63567d04-9918-426d-9d76-3d08d1a0c0c7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bsp-armbian-logo

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added 05 Milestone: Second quarter release size/medium PR with more then 50 and less then 250 lines Needs review Seeking for review BSP Board Support Packages labels Apr 14, 2026
Companion wordmark variants for /usr/share/pixmaps/armbian-logo.svg.
Mirrors upstream Ubuntu's three-file pattern in base-files:

  ubuntu-logo.svg            (logomark, picked by LOGO= os-release field)
  ubuntu-logo-text.png       (wordmark, light background)
  ubuntu-logo-text-dark.png  (wordmark, dark background)

Apps that prefer text-bearing icons (boot loaders, some greeters,
some about dialogs that pick a wider banner) look for the *-text
companions; without them they fall back to upstream branding even
when the SVG is present.

Both PNGs are 400x64 (Ubuntu's are 260x91 / 256x89; the wider
ratio matches the Armbian wordmark). 8 KB each.
@igorpecovnik igorpecovnik changed the title bsp: ship /usr/share/pixmaps/armbian-logo.svg bsp: ship Armbian distributor logo (svg + wordmark png variants) Apr 14, 2026
@igorpecovnik igorpecovnik merged commit 5b6536b into main Apr 14, 2026
11 checks passed
@igorpecovnik igorpecovnik deleted the bsp-armbian-logo branch April 14, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

05 Milestone: Second quarter release BSP Board Support Packages Needs review Seeking for review size/medium PR with more then 50 and less then 250 lines

Development

Successfully merging this pull request may close these issues.

1 participant