-
Notifications
You must be signed in to change notification settings - Fork 97
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
Ruby dependencies may be undesirable? #274
Comments
From ros/urdfdom_headers#59 (comment):
@traversaro Just to check, what would a more ideal dependency be? Pure C++, Python, a more lightweight Ruby implementation, ...? From prior conversations, I believe most Ignition tools use Ruby: |
Both C++ or Python would be much more convenient from the point of view of macOS/Windows use. Having a dependency on Ruby is relatively fine as long you just use as an interpreter for processing files. If instead you want to load in a Ruby script a plugin written in C++ you will have problems if your plugin has been compiled with a C++ toolchain different from the one with which the Ruby interpreter has been compiled. You can find a few related discussions in:
In theory you could have the same problem in Python as well, but in practice (at least on Windows) the official Python Windows interpreter uses the same C++ ABI of Visual Studio, that is the compiler used |
Gotcha, thanks for the explanation!
Is this most directly addressed by gazebosim/gz-tools#7, where |
That is correct. |
Another example in which the Ruby (build) dependency is cumbersome: conda-forge/libsdformat-feedstock#9 . TL;DR: conda-forge removed ruby from their CI machine, and now the only "clean" way forward is to add a ruby port on conda-forge that builds on Windows. |
- removes Ruby dependency (see gazebosim#274) - improvements for cpplint checks (e.g. copyright header) Signed-off-by: Bi0T1N <[email protected]>
- removes Ruby dependency (see gazebosim#274) - improvements for cpplint checks (e.g. copyright header) Signed-off-by: Bi0T1N <[email protected]>
- removes Ruby dependency (see gazebosim#274) - improvements for cpplint checks (e.g. copyright header) Signed-off-by: Bi0T1N <[email protected]>
- removes Ruby dependency (see gazebosim#274) - improvements for cpplint checks (e.g. copyright header) Signed-off-by: Bi0T1N <[email protected]>
Filing this issue based on:
RobotLocomotion/drake#13231
ros/urdfdom_headers#59 (comment)
I'm not proposing any action yet, but rather to gather use cases where Ruby is high-overhead.
Ultimately, this may get closed out as not fixable, but good to have the convo here anywho.
(I assume there are complaints on ROS Answers + Gazebo Community as well; at some point, it would be nice to skim and collect those.)
The text was updated successfully, but these errors were encountered: