Repository navigation
Logo (Builtin): add Tiny Core Linux - #2655
Merged
Merged
Conversation
CarterLi
reviewed
Oct 16, 2026
| #ifdef FASTFETCH_DATATEXT_LOGO_TINYCORE | ||
| // Tiny Core | ||
| { | ||
| .names = { "Tiny Core Linux", "TinyCore", "Tiny Core", "MicroCore" }, |
Member
There was a problem hiding this comment.
Only one entry is allowed here. The ID TinyCore (case insensitive) should only be used
Contributor
Author
There was a problem hiding this comment.
Fixed in 878aff4 — the list is now just { "TinyCore" }.
You're right that the case-insensitive match already covers both ID=tinycore and NAME=TinyCore, so the extra aliases were unnecessary (and MicroCore was speculative on my part — it isn't in the os-release of the ISO I tested).
Re-verified after the change: --list-logos reports 476) "TinyCore", and auto-detection against the real 17.1 os-release still picks the logo up with no --logo flag.
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.
Summary
Adds a built-in ASCII logo for Tiny Core Linux (visual size 21 x 10), so the distro stops falling back to the generic Linux penguin. Based on the project's own logo, where the "o" of
coreis a sphere split diagonally and labelledtiny/micro.Related issue (required for new logos for new distros)
Closes #2654
Changes
src/logo/ascii/t/tinycore.txt— new logo.$1=FF_COLOR_FG_BLUE(the distro's ownos-releaseshipsANSI_COLOR="0;34"),$2=FF_COLOR_FG_LIGHT_WHITE.src/logo/ascii/t.inc— registers it under the single nameTinyCore, in alphabetical position between TileOS and Torizon OS.TinyCorecovers bothID=tinycoreandNAME=TinyCorefromos-release.Tiny Core Linux/Tiny Core/MicroCore; those were removed per review.Screenshots
fastfetch --logo tinycore -s title:os:kernel:shell:cpu:Auto-detection with a real Tiny Core
os-releaseand no--logoflag (the check that actually matters):Confirmation that the name is registered and resolvable:
Checklist
Verification performed on this branch:
cmake -B build -DCMAKE_BUILD_TYPE=Releasefollowed bycmake --build build. Builds clean with no new warnings.--list-logoslists the new entry;--logo tinycoreresolves; auto-detection viaID=tinycorepicks it up with no flag (see output above).$1/$2placeholders only, and has no trailing spaces.Revision
t.inc:91): the logo now registers a single name,TinyCore, instead of four aliases. Rebuilt and re-verified afterwards —--list-logosreports476) "TinyCore", and auto-detection against the real 17.1os-releasestill resolves it with no--logoflag.One note for reviewers: this is a logo addition only, so
logo_builtin.his regenerated by CMake from the file glob as usual — no generator changes were needed.