Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use yaml_cpp from Ubuntu Xenial. #7352

Merged
merged 4 commits into from
Nov 3, 2017
Merged

Use yaml_cpp from Ubuntu Xenial. #7352

merged 4 commits into from
Nov 3, 2017

Commits on Nov 2, 2017

  1. Use yaml_cpp from Ubuntu Xenial. 

    This removes the bazel external yaml_cpp and replaces it with system
yaml_cpp found via pkg-config.

In order to link against it additional dependencies on @yaml_cpp needed
to be specified.

* Remove vendored yaml-cpp build config.
    @nuclearsandwich
    nuclearsandwich committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    42404f7 View commit details
    Browse the repository at this point in the history

  2. Fix yaml-cpp dependencies in automotive. 

    As part of converting the Bazel drake build to using the system yaml-cpp a few
dependencies were shuffled that should not have been.

This reverts those changes based on feedback from jwnimmer-tri.
    @nuclearsandwich
    nuclearsandwich committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    3c31f46 View commit details
    Browse the repository at this point in the history

  3. Exclude yaml_util.h header from install.

    @jwnimmer-tri @nuclearsandwich
    jwnimmer-tri authored and nuclearsandwich committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    dfa9c9f View commit details
    Browse the repository at this point in the history

  4. Add yaml-cpp link flag to installed libdrake.so. 

    Added based on feedback in RobotLocomotion#7352
This may only be a temporary measure pending maturity of the pc2cps module.
    @nuclearsandwich
    nuclearsandwich committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    59e2060 View commit details
    Browse the repository at this point in the history