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

inconsolata: revert to working unstable version #136647

Merged
merged 1 commit into from
Oct 16, 2021

Conversation

jb55
Copy link
Contributor

@jb55 jb55 commented Sep 4, 2021

An update to an unstable version of google-fonts broke the Inconsolata
monospace font in programs that use xft such as urxvt. This patch reverts
some inconsolata back to a working version.

Alternative to #130209

See also:

googlefonts/Inconsolata#42
https://discourse.nixos.org/t/how-to-fix-broken-inconsolata-font-in-urxvt-on-21-05/13437

cc @jakubgs @06kellyjac @petabyteboy @tu-maurice

Signed-off-by: William Casarin jb55@jb55.com

An update to an unstable version of google-fonts broke the Inconsolata
monospace font in programs that use xft such as urxvt. This patch reverts
some inconsolata back to a working version.

cc @jakubgs @06kellyjac @petabyteboy @tu-maurice

Signed-off-by: William Casarin <jb55@jb55.com>
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/how-to-fix-broken-inconsolata-font-in-urxvt-on-21-05/13437/6

Copy link
Contributor

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have used your branch to install Inconsolata on my desktop and it works.

Thanks for this change!

@06kellyjac
Copy link
Member

Another option would be to source from https://github.com/googlefonts/Inconsolata

Also you might want to swap to stdenvNoCC as it'll need less rebuilds

@7c6f434c
Copy link
Member

7c6f434c commented Sep 8, 2021

Maybe add a link to this «interpretation-of-monospace» issue, and a note to switch back to the unified approach once the issue gets resolved?

@jb55
Copy link
Contributor Author

jb55 commented Sep 8, 2021 via email

malloc47 added a commit to malloc47/config that referenced this pull request Sep 26, 2021
Underlying bug:

googlefonts/Inconsolata#42

As yet unmerged fix:

NixOS/nixpkgs#136647

This PR can be reverted once this fixed is merged upstream.
@jakubgs
Copy link
Contributor

jakubgs commented Oct 16, 2021

Can we get this merged? It works fine.

@SuperSandro2000 SuperSandro2000 merged commit 9d2f5ff into NixOS:master Oct 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants