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
fribidi: update source #32725
fribidi: update source #32725
Conversation
Why did you add |
It's an optional dependency. By the way, previously |
It looks like
I'll remove the dependency and add a comment saying that it is not needed. |
I see. Thanks! |
|
||
outputs = [ "out" "devdoc" ]; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = http://fribidi.org/; | ||
homepage = https://github.com/fribidi/fribidi/; |
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.
We could probably do without the trailing slash.
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.
Fixed!
0232e15
to
51f47d6
Compare
The old site redirects to GitHub.
Motivation for this change
Fixes #32709.
/cc @Fuuzetsu
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)