Skip to content
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

launch failed #21

Closed
SolarAquarion opened this issue Nov 24, 2016 · 9 comments
Closed

launch failed #21

SolarAquarion opened this issue Nov 24, 2016 · 9 comments
Labels

Comments

@SolarAquarion
Copy link

RUST_BACKTRACE=1 alacritty  101 ↵  1471  20:36:18
device_pixel_ratio: 1
thread 'main' panicked at 'assertion failed: (left == right) (left: 1, right: 0)', font/src/ft/list_fonts.rs:161
stack backtrace:
1: 0x556182a565f3 -
2: 0x556182a5bbed -
3: 0x556182a5b7eb -
4: 0x556182a5c078 -
5: 0x556182a5bf12 -
6: 0x556182a5be50 -
7: 0x556182a2697f -
8: 0x55618297e03c -
9: 0x556182a64996 -
10: 0x556182a5c8f8 -
11: 0x7f15fe914290 - __libc_start_main
12: 0x55618293ac39 -
13: 0x0 -

@jwilm
Copy link
Contributor

jwilm commented Nov 25, 2016

Thanks for the report! Have you setup an .alacritty.yml? In any case, the code here should probably handle bad font config and provide a helpful message.

@SolarAquarion
Copy link
Author

SolarAquarion commented Nov 25, 2016 via email

@jwilm
Copy link
Contributor

jwilm commented Nov 25, 2016

I hadn't thought about having a system level configuration file; that's a good idea. Per user configuration is supported by reading from $HOME/.alacritty.yml or $HOME/config/alacritty.yml. You can copy the alacritty.yml from the project root.

@SolarAquarion
Copy link
Author

it still fails

@jwilm
Copy link
Contributor

jwilm commented Nov 30, 2016

I'm guessing the font specified in the config isn't available on your system. Adding an issue to improve error messages on that front.

@SolarAquarion
Copy link
Author

SolarAquarion commented Nov 30, 2016 via email

@jwilm jwilm added the B - bug label Dec 17, 2016
@jwilm
Copy link
Contributor

jwilm commented Dec 31, 2016

Alacritty's fontconfig wrapper has been rewritten to not panic at all, and I think that means the original error you encountered should be fixed. There's still some work to do on error handling in the font loading code, however.

@SolarAquarion
Copy link
Author

it's launches now thankfully. I just need to change the font to a nicer one to be more "Nerd Font"

@jwilm
Copy link
Contributor

jwilm commented Jan 3, 2017

Awesome. So others don't run into the same issue you did, I've updated the font loading code default to find whatever monospace font is default on the system and use that.

@jwilm jwilm closed this as completed Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants