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
victor-mono: init at 1.2.1 #64544
victor-mono: init at 1.2.1 #64544
Conversation
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.
Just some nitpicks :)
in fetchzip rec { | ||
name = "victor-mono-${version}"; | ||
|
||
url = "https://github.com/rubjo/victor-mono/archive/v1.2.1.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.
be easier to update
url = "https://github.com/rubjo/victor-mono/archive/v1.2.1.zip"; | |
url = "https://github.com/rubjo/victor-mono/archive/v${version}.zip"; |
(thanks reviewer!)
It seems I've merged the competing PR, seems that this one installed Too late though. |
Yes, is there a reason to prefer one over the other?
Or to "only" include one, if upstream provides both? (space?)
Never sure which way to go with these :).
…---
Oh, I see--I'll take a look and maybe submit a PR with the difference
:).
On Sat, 13 Jul 2019 09:07:24 -0700, worldofpeace ***@***.***> wrote:
It seems I've merged the competing PR, seems that this one installed `.otf` too.
Too late though.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#64544 (comment) part: text/html
|
I'd distribute both. |
Updated, hopefully "best of both"? :) |
Motivation for this change
Fixes #64465.
Haven't tested installation but inspecting output LGTM-- requester want
to confirm this works for you? :)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)