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.

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

mrt_cmake_modules: 1.0.10-1 in 'noetic/distribution.yaml' [bloom] #42231

Merged
merged 1 commit into from
Jul 28, 2024

Conversation

poggenhans
Copy link
Contributor

Increasing version of package(s) in repository mrt_cmake_modules to 1.0.10-1:

mrt_cmake_modules

* FindGeographicLib: Fix for GeographicLib 2.* and Windows
  Since GeographicLib version 2, the library name changed from libGeographic.so to libGeographicLib.so, see https://github.com/geographiclib/geographiclib/blob/5e4425da84a46eb70e59656d71b4c99732a570ec/NEWS#L208 .
  To ensure that GeographicLib 2.* is found correcty, I think we should add also GeographicLib to the names used by find_library.
  Furthermore, on Windows the import library is called GeographicLib-i.lib (see https://github.com/geographiclib/geographiclib/blob/v2.3/src/CMakeLists.txt#L119), so to find the library correctly on Windows we also look for GeographicLib-i .
* add ortools
* Revert "mrt_add_library now adds a compilation tests for all headers used by the library"
  This reverts commit b05cac0200ce6b8de8e8a18789dbd58cd9d8d1eb.
* Merge branch 'master' into HEAD
* Changes how the check for formatting is done.
  Now the CI job uses the --check flag provided by cmake_format instead of
  the git diff check, because git caused some problems in this repo.
* format
* mrt_add_library now adds a compilation tests for all headers used by the library
* Add ZeroMQ
* Add zxing-cpp to cmake.yaml.
* hard coded ignore files which start with "mocs_compilation and delete the corresponding gcda file, because otherwise our current coverage pipeline fails.
* Contributors: Fabian Poggenhans, Jan-Hendrik Pauls, Johannes Beck, Kevin Rösch, Mrt Builder, Yinzhe Shen

@github-actions github-actions bot added the noetic Issue/PR is for the ROS 1 Noetic distribution label Jul 27, 2024
@mjcarroll mjcarroll merged commit b31124f into ros:master Jul 28, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
noetic Issue/PR is for the ROS 1 Noetic distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants