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
sisco.lv2: init at 0.7.0 #20868
sisco.lv2: init at 0.7.0 #20868
Conversation
Also added maintainer `e-user`
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.
Can you switch the sources to use fetchFromGitHub instead? Otherwise all the rest is looking good
robtkVersion = "80a2585253a861c81f0bfb7e4579c75f5c73af89"; | ||
robtkName = "robtk-${robtkVersion}"; | ||
|
||
src = fetchurl { |
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.
for github archives it's better to use fetchFromGitHub
That change didn't go as smooth at it ought to be, but there you go. |
The CI error looks completely bogus to me. |
|
||
let | ||
name = "sisco.lv2-${version}"; | ||
name = "sisco.lv2-${src.rev}"; |
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.
the previous change was correct. The version shouldn't be prefixed with "v", it breaks the update detection otherwise.
Almost there! |
What's left to do, now? |
All good now :) |
* sisco.lv2: init at 0.7.0 Also added maintainer `e-user` * Use fetchFromGitHub and adapt accordingly * Use `version` instead of `src.rev` in name
Motivation for this change
I needed sisco.lv2 myself for analyzing waveforms.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)Also added maintainer
e-user