-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
New package: colmap-3.8 #42457
New package: colmap-3.8 #42457
Conversation
srcpkgs/metis/template
Outdated
build_style=gnu-makefile | ||
hostmakedepends="cmake" |
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.
why not just use the cmake build style?
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.
Even though this has a custom do_configure()
? If that's okay, then sure.
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 shouldn't need to touch their weird makefile -- it runs cmake internally
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 makefile does do one important thing - resolve the absolute path the the GKlib
folder. If that isn't passed to CMake, then I get configure errors.
Could have been done within CMake, but doing it ourselves isn't too bad.
f3567bb
to
17d43b5
Compare
17d43b5
to
17ff194
Compare
Closing since I am no longer using it and don't want to maintain it. If somebody else wants to pick this up, feel free. |
Testing the changes
New package