Skip to content

Conversation

@Bering
Copy link

@Bering Bering commented Jul 11, 2020

Hi,

Please add support for the Manjaro logo. I could not figure out how you auto-detect the logo so this pull request only has the logos in it. Sorry, my C is really rusty!

Thanks for this software, I like it! No dependencies, straight to the point, great! :-)

Logo:

██████████████████  ████████
██████████████████  ████████
██████████████████  ████████
██████████████████  ████████
████████            ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████
████████  ████████  ████████

Banner:

██████████  ████
██████████  ████
██████████  ████    __  __              _                   _     _                  
████        ████   |  \/  | __ _ _ __  (_) __ _ _ __ ___   | |   (_)_ __  _   ___  __
████  ████  ████   | |\/| |/ _` | '_ \ | |/ _` | '__/ _ \  | |   | | '_ \| | | \ \/ /
████  ████  ████   | |  | | (_| | | | || | (_| | | | (_) | | |___| | | | | |_| |>  < 
████  ████  ████   |_|  |_|\__,_|_| |_|/ |\__,_|_|  \___/  |_____|_|_| |_|\__,_/_/\_\
████  ████  ████                     |__/                 

@deater
Copy link
Owner

deater commented Aug 5, 2020

linux_logo doesn't auto-detect distribution, it's up to the distribution to put an /etc/linux_logo.conf that sets the default logo to use.

your patch is using unicode characters. For the ASCII logos they should print properly on a 7-bit ASCII display so you can't use unicode.

Unicode might be OK for the color logos, though it's preferable not to. You can generate a colored box by using a space with a background color set, but that might only let you have darker colors on most terminals.

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.

3 participants