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
openmvs: bump version to latest release and fix build #98134
Conversation
@@ -57,8 +56,7 @@ stdenv.mkDerivation { | |||
homepage = "http://cdcseacave.github.io/openMVS/"; | |||
license = licenses.agpl3; | |||
platforms = platforms.linux; | |||
maintainers = with maintainers; [ mdaiter ]; | |||
maintainers = with maintainers; [ mdaiter timput ]; | |||
# 20190414-174115: CMake cannot find CGAL which is passed as build input |
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.
Is the comment still relevant if this is not broken anymore?
Also, I can verify that this builds (which I did), but I don't really know what this does, so I can't really test it.
Seeing as you're adding yourself as maintainer, I'd trust you that this actually does what it's supposed to.
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.
You are correct about that comment, I've removed it now.
I've also changed the commit message to better align with CONTRIBUTING.md
d62955e
to
48b2aab
Compare
48b2aab
to
8327715
Compare
@ofborg eval |
Please fix evaluation
|
I marked this as stale due to inactivity. → More info |
Closed by #211780. |
Motivation for this change
openmvs-unstable was based on an old version of openmvs and has been broken for a long time.
Things done
This commit bumps openmvs to a stable release, bumps the CGAL dependency to a needed newer version and fixes the build.
cdcseacave/openMVS#364 (comment)
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)