-
Notifications
You must be signed in to change notification settings - Fork 118
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
Top-level Windows installation instructions #96
Comments
I do not know what is the strategy for installing Windows dependencies, but I am currently experimenting try to install all the dependencies of Ignition Dome on Windows via conda-forge. Install Visual Studio with C++ support and a conda distribution such as Miniconda.
Then, activate the environment in any terminal that you open related to the Ignition compilation:
Install the dependencies:
Create a colcon workspace in the user home:
Go back to the workspace root and check that colcon sees the workspace:
Build:
|
Thank you for the pointers, @traversaro ! I believe we were going to take the vcpkg approach. @j-rivero / @JShep1 , what do you think of giving conda-forge a try? |
I think that both vcpkg and conda-forge on Windows at the moment are quite blocked by dartsim/dart#1522. In addition, vcpkg is also blocked by gazebosim/gz-tools#7, while conda-forge (in theory, there are a few bugs around) at least has a Ruby binary compatible with MSVC. |
@chapulina it would be great if you give conda-forge a try. As a bonus, you can generate native installers for Gazebo easily using the conda constructor. That should make it quite straightforward for your users to install gazebo quickly: https://github.com/conda/constructor I am happy to help wherever I can wrt. conda-forge. |
We've been putting together some docs internally for building |
@traversaro in response to your previous comment/guide for
I'm running VS 2019 and installed package dependencies up to
are you seeing this error at all with VS 2017 or possibly know what might be going on here? I see |
I think this is related to this patch in ignition-cmake in https://github.com/conda-forge/libignition-cmake0-feedstock/blob/master/recipe/FindGTS.cmake.patch, sorry for that. I am not sure why this issue is not emerging in vcpkg instead, probably glib there installs its header in another directory. I can check that. |
By looking a bit more, I think that https://github.com/conda-forge/libignition-cmake0-feedstock/blob/master/recipe/FindGTS.cmake.patch (originally provided by @seanyen in https://github.com/conda-forge/libignition-cmake0-feedstock/pull/2/files) is not a conda-specific patch, and should just be added to ignition-cmake . The fact that the glib.h headers are installed in
For this reason, I think it make sense that the fallback CMake logic should find and provide this include directories. |
More long term solution:
|
@traversaro Awesome, thanks for the quick and detailed response. I'm going to try to get a local temporary workaround going so I'm not blocked, and then try to get a PR in for |
I would first open an issue in https://github.com/conda-forge/gts-feedstock/issues to track the enhancement of adding .pc files for gts on Windows (similar to microsoft/vcpkg#11808 in vcpkg). Then the gts feedstock on Windows is using a custom CMakeLists to build the library, so the .pc file generation should be added in there: https://github.com/conda-forge/gts-feedstock/blob/master/recipe/CMakeLists.txt#L95 . For general info, you can check contributing docs in conda-forge https://conda-forge.org/docs/user/contributing.html and general info on how conda build builds recipe https://docs.conda.io/projects/conda-build/en/latest/concepts/recipe.html . |
We're going with the conda-forge approach for the tutorials for now due to its convenience. But it's important to keep in mind that our CI is using vcpkg, so there's a great chance that the tutorials will regress from time to time. Here's an issue about potentially moving our CI to conda-forge: gazebo-tooling/release-tools#374 |
Much of the stack already works on Windows, but we don't have documentation explaining how to install it. We should add:
Some useful resources:
The text was updated successfully, but these errors were encountered: