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
ocaml-libvirt: 0.6.1.4.2017-11-08-unstable -> 0.6.1.5 #102533
Conversation
src = fetchFromGitLab { | ||
owner = "libvirt"; | ||
repo = "libvirt-ocaml"; | ||
rev = "refs/tags/v${version}"; |
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.
rev = "refs/tags/v${version}"; | |
rev = "v${version}"; |
Shouldn't this just be the long of my change?
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.
Thanks, I didn't realize it works for tags with fetchFromGitlab
.
Result of 1 package failed to build:
5 packages built:
|
What about using the sources from there: https://libvirt.org/sources/ocaml/ocaml-libvirt-0.6.1.5.tar.gz? |
It seems fix for Another small change for
I'm fairly new to contributing to nixpkgs, so I don't know - whatever is usually preferred (is there some policy on using git vs tarballs, or Github/Gitlab vs project websites?) Normally I prefer Github/Gitlab for any sources because they are less likely to go down or change domain or other part of URL. |
Result of 3 packages marked as broken and skipped:
|
@GrahamcOfBorg build libguestfs vagrant virt-top |
Motivation for this change
Fix a few problems and bumps it to the latest version:
git://git.annexia.org/git/ocaml-libvirt.git
was not working, it should have beengit://git.annexia.org/ocaml-libvirt.git
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)