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

Also include buildtool-depends in find_package(catkin ...) #31

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Also include buildtool-depends in find_package(catkin ...) #31

wants to merge 5 commits into from

Commits on Mar 14, 2016

  1. Consider both build_depend and buildtool_depend as potential catkin b… 

    …uild depends.
    @NikolausDemmel
    NikolausDemmel committed Mar 14, 2016
    Configuration menu
    Copy the full SHA
    103f0c4 View commit details
    Browse the repository at this point in the history

  2. whitespace

    @NikolausDemmel
    NikolausDemmel committed Mar 14, 2016
    Configuration menu
    Copy the full SHA
    bbacbed View commit details
    Browse the repository at this point in the history

Commits on Mar 28, 2016

  1. various improvements with format 2 and pure cmake package support 

    Changes include:

- Packages found with `find_package` that are no cmake packages are now used in
  `include_directories` and automatically linked to, using the standard
  `..._INCLUDE_DIRS` and `..._LIBRARIES` variables. Moreover, they are exported
  by being passed as `DEPENDS` in `catkin_package`.

- Buildtool-depends are now also find-packaged.

- `cs_export` now also has an option flag-argument `ALL_DEPS_REQUIRED`.

- `cs_export` uses build(tool)-export depends, not run-depends to determine what
  is exported in the `catkin_package` call.

- There is now an informative message about build(tool)-(export) depends that
  are listed in package.xml, but not found with `find_package`. This makes is
  easier to track down typos in catkin package names.
    @NikolausDemmel
    NikolausDemmel committed Mar 28, 2016
    Configuration menu
    Copy the full SHA
    ef626e1 View commit details
    Browse the repository at this point in the history

  2. Update package.xml to format 2.

    @NikolausDemmel
    NikolausDemmel committed Mar 28, 2016
    Configuration menu
    Copy the full SHA
    b30b6f2 View commit details
    Browse the repository at this point in the history

  3. fix variable expansion

    @NikolausDemmel
    NikolausDemmel committed Mar 28, 2016
    Configuration menu
    Copy the full SHA
    4dd4c2e View commit details
    Browse the repository at this point in the history