-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
ttf_envy_code_r: init at preview7 #27165
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
Conversation
It seems that the license is not free. Below the download link, on can read:
|
So, how about changing the license to |
@wheatdog Best thing is to just ask the author of the package to change the wording of what he said, because it's rather vague. |
I believe it's clear that the font is non-redistributable and therefore unfree. I think it can be included in nixpkgs, with |
@wheatdog, if you squash those two commits, the PR will be ready to be merged. However, I am not sure if the license allows redistributing this font, as Hydra will inevitably package it and provide a binary package. Or is there a way to deactivate it for this one package? Maybe by setting |
@jfrankenau That's what the unfree is supposed to be for. Or are you saying that despite it being marked as unfree that it would still be built? |
pkgs/top-level/all-packages.nix
Outdated
@@ -13096,6 +13096,8 @@ with pkgs; | |||
|
|||
ttf_bitstream_vera = callPackage ../data/fonts/ttf-bitstream-vera { }; | |||
|
|||
ttf_envy_code_r = callPackage ../data/fonts/ttf-envy-code-r {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any particular reason for underscores? Dashes are preferred nowadays: http://nixos.org/nixpkgs/manual/#sec-package-naming
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I prefer dahes, too. I will fix it.
@0xABAB, ah I didn't know that setting |
Some distros, e.g. one pseudo-fork, don't even allow unfree stuff into the main repo, typically for "ideological reasons" :-) If we distributed non-distributable packages, that could be quite a legal problem. |
That's why I wanted to make sure that this package is only built by the end user. And I realized that NixOS being source-based and only backed by binary caches is ideal for this kind of situation. |
I am not sure how nixpkg handle unfree packages. Can someone point out some code or document that explaining the behavior when specifying |
@wheatdog: this, for example: http://nixos.org/nixpkgs/manual/#sec-allow-unfree |
@vcunat: thanks, what about the concern @jfrankenau mentioned? Will it be cached in binary form (by Hydra, maybe)? |
Unfree packages aren't touched by Hydra, of course. Not even the redistributable ones ATM. |
@wheatdog, another section in the Nixpkgs manual implies this: https://nixos.org/nixpkgs/manual/#sec-meta-license |
OK, I'd push this with some nitpicks atop: 4227b49. Any other ideas? |
@vcunat: Looks great, but why do you change the location of readme? Will it collide? The reason I put it that way is that terminus-font-ttf seems do the same thing. |
Yes, if two packages define the same path, they will collide when getting linked into the same environment. |
(cherry picked from commit c057098) It just adds a new package.
Merged to master and 17.03. I just switched |
(cherry picked from commit c057098) It just adds a new package.
Motivation for this change
I want to use Envy Code R.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
share/fonts/truetype