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: 1.010 -> inherit google-fonts #31960

Merged
merged 2 commits into from Nov 23, 2017
Merged

Conversation

mikoim
Copy link
Contributor

@mikoim mikoim commented Nov 23, 2017

Motivation for this change

The upstream moved to Google fonts repository on GitHub.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

repo = "fonts";
rev = "4c3e95c802f8f12b78869ff50d552014de63f9c1";
sha256 = "1ndmsf4c0k36dakmps0vr7hhg5ss8m7ywja7v55xdrinvli58v2f";
};
Copy link
Member

@Mic92 Mic92 Nov 23, 2017

Choose a reason for hiding this comment

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

We have a package called google-fonts. Maybe we can take the source/version from there?

inherit (google-fonts) src version;

Do we need a dedicated inconsolata font package then at all?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I missed it. I think that it's nice to have the individual package regarding backward compatibility and package size. (google-fonts is bigger than the individual package.) If it's complicated to maintain an individual package, there is another method that split google-fonts into multiple packages (e.g., google-fonts: install all fonts, google-fonts-foobar: install foobar font only).

To be honest, I don't know whatever method is optimal.

@mikoim mikoim changed the title inconsolata: 1.010 -> 2.001 inconsolata: 1.010 -> inherit google-fonts Nov 23, 2017
@mikoim
Copy link
Contributor Author

mikoim commented Nov 23, 2017

For the time being, inherit source and version from google-fonts.

@Mic92 Mic92 merged commit e645ecf into NixOS:master Nov 23, 2017
@mikoim mikoim deleted the fonts/inconsolata branch November 24, 2017 03:01
@jb55
Copy link
Contributor

jb55 commented Dec 18, 2017

I'm trying to find out why my fonts look garbage after my latest update to the unstable channel.... perhaps it was this. I will try to revert and report back.

@Mic92
Copy link
Member

Mic92 commented Dec 18, 2017

volth wrote:

Why this is removed?
It is here for #27754

@volth if somebody updates google-fonts this derivation might break. Updating derivations should not have unexpected side effects in other places.

@mikoim
Copy link
Contributor Author

mikoim commented Dec 19, 2017

@jb55 Sorry for the inconvenience. What issue happened in your environment?

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.

None yet

4 participants