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

Revert google-fonts update to unstable, fixing Inconsolata #130209

Closed
wants to merge 3 commits into from

Conversation

jb55
Copy link
Contributor

@jb55 jb55 commented Jul 14, 2021

Motivation for this change

An update to an unstable version of google-fonts broke the Inconsolata monospace font in programs that use xft such as urxvt. This PR reverts some recent google-fonts commits back to the stable version.

See:

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

@r-rmcgibbo
Copy link

r-rmcgibbo commented Jul 14, 2021

Result of nixpkgs-review pr 130209 at 0ffd2b9 run on x86_64-linux 1

2 packages built successfully:
  • inconsolata
  • ricty
1 suggestion:
  • warning: missing-phase-hooks

    installPhase should probably contain runHook preInstall and runHook postInstall.

    Near pkgs/data/fonts/inconsolata/default.nix:8:3:

      |
    8 |   installPhase = ''
      |   ^
    

Result of nixpkgs-review pr 130209 at 0ffd2b9 run on aarch64-linux 1

1 package built successfully:
  • inconsolata
1 suggestion:
  • warning: missing-phase-hooks

    installPhase should probably contain runHook preInstall and runHook postInstall.

    Near pkgs/data/fonts/inconsolata/default.nix:8:3:

      |
    8 |   installPhase = ''
      |   ^
    

Copy link
Member

@06kellyjac 06kellyjac left a comment

Choose a reason for hiding this comment

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

I'm against this for 2 reasons

  1. It's not just reverting the src but also reverting improvements like setting it to stdenvNoCC (which inconsolata/default.nix should use too)
  2. The inconsolata/default.nix can just change it's own src to refer to an older working commit of google/fonts rather than reverting every other font in the google-fonts repo to 2019

Edit: also calling the previous dated version stable is not true, google/fonts doesn't have any stable releases, every dated release of google/fonts is unstable. It's just someone changed the version text to make that clear

@jb55
Copy link
Contributor Author

jb55 commented Jul 14, 2021

ah good point on just updating inconsolata/default.nix, will open a new PR

@jb55 jb55 closed this Jul 14, 2021
@jb55 jb55 deleted the fix-inconsolata branch July 14, 2021 20:39
@06kellyjac
Copy link
Member

Thanks @jb55 🎉

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.

3 participants