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
nixos/fontconfig-penultimate: disable by default #67529
Conversation
I'm also curious if we should remove |
Oop, I should document this. |
I'm not too familiar with fonts in Linux, but I suspect we want either fontconfig-penultimate or fontconfig-ultimate enabled to get things working correctly. Some feedback from @ttuegel or anyone else who knows how fonts are supposed to work would be helpful. My understanding is that fontconfig-penultimate is probably a good thing to have, it's just gotten out of date in the few years since its last release. This is a little bit too close to a release for my liking, but I suppose we can always revert it in the release-19.09 branch if it causes issues. |
Sure, I totally understand if it's cutting it close. My issue currently with It would be helpful if someone who's familiar with the current state of fontconfig offered some insights. |
Cc @volth as resident font guru. |
92cbf30
to
11bd3b6
Compare
I have been running with penultimate disabled for last week or so and did not encounter any issues. The current default is much worse for me as it lacks support for both emoji and math font classes. I though we no longer patch freetype, as the main contention (ClearType hinting) was incorporated into upstream freetype when patents expired.
If someone still wants it they can always enable it. But since vanilla fontconfig is perfectly usable, as evidenced by other distros running vanilla fontconfig, I say there is no place for penultimate to be enabled by default. |
It currently lacks an emoji font-family which means it has to be disabled for them to function [0]. Additionally it's fallen out of necessity to ship custom font rendering settings (as far as I'm aware of). [0]: NixOS#67215
2d2f761
to
0d220e4
Compare
It seems we came to a good agreement during nixos office-hours, I'm going to merge as is. |
It currently lacks an emoji font-family which means it has to be
disabled for them to function 0. Additionally it's fallen out of
necessity to ship custom font rendering settings (as far as I'm aware
of).
Motivation for this change
cc @ttuegel, I'd very much appreciate your take on this.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)