RobotLocomotion / drake

Model-based design and verification for robotics.
https://drake.mit.edu
Other
3.28k stars 1.26k forks source link

Include all example binaries in the Drake install #4224

Closed david-german-tri closed 3 years ago

david-german-tri commented 7 years ago

This matters, for instance, to KUKA users.

cc @patmarion

jamiesnape commented 7 years ago

Also useful for #1766.

patmarion commented 7 years ago

I would still like to see this happen in the CMake build. But if there's no plans to do so, please close this issue.

jwnimmer-tri commented 7 years ago

I think the issue is still valid even some CMake listfiles are going away -- the Bazel binary release could / should include those artifacts.

david-german-tri commented 7 years ago

I am going to bump the priority down to medium, though.

EricCousineau-TRI commented 6 years ago

Closing, given that Francois's PRs for a subset of examples have landed.

jwnimmer-tri commented 6 years ago

I thought the goal was to install ~all examples?

jamiesnape commented 6 years ago

It is.

jwnimmer-tri commented 4 years ago

We don't seem to be suffering for not having this since the past four years, and adding more C++ binaries would only increase the image footprint when most users just want the library, not the demos. Using online notebooks for the demos seems to be the way of the future.

Unless anyone speaks up about this in the next month or two, we'll plan to close this as "won't fix".