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
iwona (font): init at 0_995 #31067
iwona (font): init at 0_995 #31067
Conversation
Just out of curiosity, can someone clarify what the flags that have been added to this issue mean? |
@siddharthist it means it affects rebuilds of no more than 10 packages. |
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.
This looks good to me. Can you remove that extra comment above the version info?
meta = with stdenv.lib; { | ||
description = "A two-element sans-serif typeface, created by Małgorzata Budyta"; | ||
homepage = http://jmn.pl/en/kurier-i-iwona/; | ||
# "[...] GUST Font License (GFL), which is a free license, legally |
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.
I don't think we need this particular comment above the license. If they say it's legally equivalent, I think we can use the lppl you are using.
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.
Somebody stumble over the difference, so I would keep the remark also there is technically no difference.
pkgs/data/fonts/iwona/default.nix
Outdated
version = "0_995"; | ||
|
||
src = fetchzip { | ||
url = "jmn.pl/pliki/Iwona-otf-${version}.zip"; |
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.
Maybe you should explicitly specify http://
protocol here for clarity ?
Thanks for the reviews, and special thanks to @Mic92 for fixing it up! I'm glad to learn about |
Motivation for this change
Another great sans-serif font!
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)