-
-
Notifications
You must be signed in to change notification settings - Fork 654
Logo (Builtin): adds LimeOS logo #2137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
Adds support for LimeOS, a Debian-based distribution. - Adds ASCII logo (white color scheme) - Adds OS detection for `ID=limeos` Homepage: https://limeos.org
|
Just noticed in As for the names in In case any maintainer / contributor sees this, please correct me if I'm wrong. I also noticed the logo filename is |
|
Please file a logo request in Github Issues |
Yea i forgot to change the file name of logo i will change when i am free |
Ok |
src/detection/os/os_linux.c
Outdated
| // LimeOS - Debian-based | ||
| ffStrbufSetStatic(&result->idLike, "debian"); | ||
| return true; | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is necessary only if ID in /etc/os-release is set as debian, otherwise you should not touch this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok sir
src/logo/ascii/lime.txt
Outdated
| @@ -0,0 +1,23 @@ | |||
| $1 ############ | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$1 is not needed.
Adds support for LimeOS, a Debian-based distribution. Homepage: https://limeos.org
added changes from fastfetch
|
Dear CarterLi, i have looked into my mistake and found the issue and according to your and LimeOS founder guidance i have resolved the issues. I apologize for my mistake. |
src/logo/builtin.c
Outdated
| }, | ||
| // LimeOS | ||
| { | ||
| .names = {"LimeOS", "limeos"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Names are matched case-insensitively, so limeos can be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok sir i didn't know that
|
@CarterLi quite early for this distro; incomplete website, no iso to download |
Just got notified about this. Feel free to either accept or deny this for the time being. We can always come back later. For context, I understand your goal is to prevent spam and the accumulation of projects that may later be abandoned. While LimeOS is still in active development, it is neither low-quality nor a throwaway project. LimeOS Ltd is a registered non-profit company in the United Kingdom, and the LimeOS name is protected by trademarks in both the European Union and the United States. Projects that take these steps typically demonstrate long-term intent and accountability - comparable to established distributions such as Debian, Arch Linux, or Ubuntu. In that sense, LimeOS already represents a significantly lower “spam risk” than the majority of listed or supported distributions. |
|
@FabioLolix Thanks for your opinion. What about to set a hard requirement: any new distros that want to add their logo into fastfetch, must have downloadable ISOs and must have a complete webside ( can be simple though ) |
As for the downloadable ISO, since yesterday there is a prototype |
Adds support for LimeOS, a Debian-based distribution.
ID=limeosHomepage: https://limeos.org