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
[laszip] Migrate LASzip to the new manifest format. #18614
[laszip] Migrate LASzip to the new manifest format. #18614
Conversation
@CreMindES ,Thanks for your pr . |
- CONTROL -> vcpkg.json - vcpkg_configure_cmake -> vcpkg_cmake_configure - vcpkg_install_cmake -> vcpkg_cmake_install
6c27fe3
to
9f76507
Compare
@JonLiu1993 Could you elaborate on the motivation behind that or point me to the a docs about it? I've looked but did not find much. |
I didn't find the related changes about this, can you confirm that? |
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.
LGTM
Description
What does your PR fix?
Migrate LASzip to the preferred manifest format.
Does your PR follow the maintainer guide?
Yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --all
and committed the result?Yes
Note
It also fixed an issue on Windows with MSVC where
LASZIP_ROOT
needed to be defined. With the new cmake configure and install functions this is no longer needed.