Skip to content

Cleaned up CMake#48

Merged
amock merged 14 commits intouos:developfrom
amock:better-cmake
Nov 23, 2025
Merged

Cleaned up CMake#48
amock merged 14 commits intouos:developfrom
amock:better-cmake

Conversation

@amock
Copy link
Member

@amock amock commented Nov 22, 2025

This cleans up the installation part of the CMake scripts. After following the migration guide (I added to this repo), I was able to run MeshNav successfully with these installation types of lvr2:

  • global (/usr/local)
  • local ROS 2 ws installation (ros_ws/install)
  • local package installation (pre-compiled libraries) (see packaging_notes.md)

Since I changed the cmake interface, I increased the minor version. Maybe we can get fully downwards compatible before merging this (to version 25.1.0).

This change is important to support precompiled binaries of lvr2. I already put a working runner into the repo. It works like this:

  • We merge this into the main branch
  • We create a release 25.2.0
  • Then a automatic job is triggered that is building and packaging everything and updates the release

@amock amock requested a review from JustusBraun November 22, 2025 00:46
@amock amock self-assigned this Nov 22, 2025
Copy link
Contributor

@JustusBraun JustusBraun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Depending projects compiled without any changes

… of CMakeLists to an extra cmake script. added core dependencies to cmake
@amock amock merged commit fe3c474 into uos:develop Nov 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants