-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Font looks different + really bad kerning #16978
Comments
My guess is 1e53d4a. cc @ericsagnes. I'll try to switch to master and debug what happens. |
Wild guess, is there a Could you post the result of |
|
Hm, I upgraded another machine to the same commit. Fonts look generally a bit different than they used to (for instance, in chromium + gmail). (I think the glyphs are just thinner and there is a lot of "air" around each letter.) But the horribly broken stuff in gnome-terminal doesn't happen on this machine. What's going on? |
Can you try to remove |
Yes it does! Now all text look like it used to. Thanks!
After this, /etc/fonts.tmp is no more, only /etc/fonts:
|
Is this something we can fix in the NixOS module that sets up /etc/fonts, so that users upgrading from 16.03 to 16.09 won't hit this? |
A heuristic idea for this -- a function
On any conflicts, if old path is static, it's safe to replace and/or delete stale. Otherwise make a backup and notify the user via a journal entry and console output. What are the possible caveats? This would certainly be nice to implement. I can look into this if it seems a good idea, but I suspect corresponding code is in Perl, which I'm illiterate with. On July 15, 2016 1:25:53 PM GMT+03:00, "Bjørn Forsman" [email protected] wrote:
Nikolay. |
@abbradar Yes, the cause seems to be in the |
Truth be told, I like better an idea of making |
@abbradar in principle I'd like that, but in practice FHS is pretty unclear about whether /etc is supposed to be mutable. Would definitely be interested in what you came up with though. Personally, I'd rather fix more programs to not use /etc and rather to link directly to configuration in the store that they need (since we get more granularity there and know exactly what needs to be mutable), but that's probably a lot of work in some cases. |
For input method packages, I used dedicated packages for the Could a similar approach for |
@copumpkin, why would we care what FHS says? :-D |
I get this line when switching (and then fonts look ugly):
Where 81 line is https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/system/etc/setup-etc.pl#L81. Next fixes the fonts: sudo rm /etc/fonts*
sudo nixos-rebuild switch |
Opened a PR with a fix: #17042. |
@rasendubi Nice find! I haven't yet done any research, so this thread will come handy. |
This patch adds handling of a directory becoming a symlink in /etc. Before this patch, the directory wasn't removed and then symlinking failed, which caused directory not being updated at all. The idea for the patch goes to @abbradar at NixOS#16978 (comment): > A heuristic idea for this -- a function `isStatic :: Path -> Bool`: > > * if path `/etc/foo` is a file, return True iff it's a symlink to `/etc/static/foo`. > * if path is a directory, return True iff for all items in it `isStatic` is True. > > On any conflicts, if old path is static, it's safe to replace and/or > delete stale. Otherwise make a backup and notify the user via a > journal entry and console output. The only difference here -- it will not replace user configs. This also fixes NixOS#16978.
Issue description
Somewhere between b5a95b1 and 6af0f57 there is a commit that changes the default fonts on NixOS. Most text is just different (I'd say, a bit less readable). But some text in the terminal (gnome-terminal) is completely borked (in regards to positioning):
Look at how the 's' is drawn on top of the 'M'. And generally the positioning of glyphs is... not right!
Steps to reproduce
Compare how fonts look in a few apps on nixpkgs b5a95b1 vs 6af0f57.
Technical details
The text was updated successfully, but these errors were encountered: