vollkorn: init at 4.105

[tfmoraes]

Motivation for this change

Add Vollkorn typeface in NixOS.

Things done

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
• 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 nixpkgs-review --run "nixpkgs-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Ensured that relevant documentation is up to date
• Fits CONTRIBUTING.md.

[sternenseemann]

Vollkorn is also included in google-fonts. Not sure what is the best way to unify this; having the bundle is useful, but probably not everybody wants to install all google fonts.

[SuperSandro2000]

@ofborg eval

[SuperSandro2000]

Result of nixpkgs-review pr 96860 run on x86_64-linux 1

1 package built:

• vollkorn

[tfmoraes]

@SuperSandro2000 I accepted you modification but I had to remove your modifications because it was not working.

[SuperSandro2000]

@SuperSandro2000 I accepted you modification but I had to remove your modifications because it was not working.

Because it is no a mkderivation.

[tfmoraes]

Other fonts don't use mkDerivation too, like https://github.com/NixOS/nixpkgs/blob/ac19435d99ebd8d5ee52abc0a268f387300a04ca/pkgs/data/fonts/sudo/default.nix and https://github.com/NixOS/nixpkgs/blob/683a87dbeb5cbf7a97fc8093836d3691e79780a1/pkgs/data/fonts/cozette/default.nix

[SuperSandro2000]

Yeah, you are right.

[xaverdh]

Is this obsolete now with #117096 merged?

[tfmoraes]

@xaverdh I'll remove this PR.