-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
How to solve the following problems when create new ports? #834
Comments
Hi!
Hopefully this makes it clearer that the message is part of the post-build validation. |
Thanks! For the third problem, after looking into the toolsrc, I think there is no workaround without add additional options in these lines. I hope I'm wrong, if not, maybe adding some options in
|
One option is to move the plugins to the file(MAKE_DIRECTORY ${CURRENT_PACKAGES_DIR}/tools/openscenegraph)
file(RENAME ${CURRENT_PACKAGES_DIR}/debug/bin/osgPlugins-3.5.5 ${CURRENT_PACKAGES_DIR}/tools/openscenegraph/plugins) This way, the core library (which is expected to be linked into the user's application) will still be fully checked. However, we generally don't have a strong story for libraries with a plugin architecture today because you can't make a choice at compile time; you have to choose at deployment time. We'd like to make this a better experience in the future! |
…nd *.bat) to `tools/` otherwise I get `POST_BUILD_CHECKS_FAILED`. I followed this recommendation: microsoft#834 (comment) Now the *.bat files need to be fixed with the correct path to `tools/`
* [colmap] Add port for COLMAP 3.6-dev.3 * Fixed some errors in portfile * Fixing some errors in portfile. These changes also move the binary (and *.bat) to `tools/` otherwise I get `POST_BUILD_CHECKS_FAILED`. I followed this recommendation: #834 (comment) Now the *.bat files need to be fixed with the correct path to `tools/` * Update ports/colmap/portfile.cmake Co-authored-by: Robert Schumacher <[email protected]> * Update ports/colmap/portfile.cmake Co-authored-by: Robert Schumacher <[email protected]> * Update ports/colmap/usage Co-authored-by: Robert Schumacher <[email protected]> * First working version, the line `vcpkg_copy_tools(TOOL_NAMES colmap AUTO_CLEAN)` helped. Now it does work running: > <vcpkg-root>\vcpkg\packages\colmap_x64-windows\tools\colmap\colmap.exe > <vcpkg-root>\vcpkg\packages\colmap_x64-windows\tools\colmap\colmap.exe gui ToDo: use `vcpkg.json`. * Using `vcpkg.json` instead of CONTROL file * Formatting json file: ./vcpkg.exe x-format-manifest --all * Changes suggested by @ahojnnes (colmap author). Waiting for the 3.7 release (probably today) to update the `REF` and `SHA512` * Changes proposed from @ahojnne to handle the internal colmap variables `DGIT_COMMIT_ID` and `DGIT_COMMIT_DATE` * Enabling TESTS (changes sent by @ahojnnes) * Updated to lastest tag to 3.6 * Updating to 3.6 * Ceres was missing suitesparse/lapack * Update ports/colmap/portfile.cmake (commit suggested by JackBoosY) Co-authored-by: Jack·Boos·Yu <[email protected]> * format manifest * Update ports/colmap/portfile.cmake Co-authored-by: nicole mazzuca <[email protected]> Co-authored-by: Robert Schumacher <[email protected]> Co-authored-by: JackBoosY <[email protected]> Co-authored-by: Jack·Boos·Yu <[email protected]> Co-authored-by: nicole mazzuca <[email protected]>
Hi,
I would like to add a new port following this tutorial, the library to be ported is OpenSceneGraph(osg), which is also cubersome to be built for new comers. But vcpkg has all the dependencies and it is quite easy to be ported.
The port is successfully configured and built, but I have some post-built problems as below.
1, To remove installation of includes for debug. There is the following warning, I think it is easy to handle as described.
2, To not install exe files to bin. OSG has some handy apps to be built, but how to install them? Absolutely, I could disable by
-BUILD_OSG_APPLICATIONS=OFF
in the profile.cmake, but how to install them? Because these apps are also often used.installed/x64-windows
folder.Is there are some hints on how to solve this? Or direct me to some existing ports with solution to similar problem?
Thanks,
Han
The text was updated successfully, but these errors were encountered: