Skip to content

Unify CMake patterns and add cheatsheet - #1

Merged
mmmarinho merged 4 commits into
mainfrom
unify-cmake-patterns
Aug 15, 2026
Merged

Unify CMake patterns and add cheatsheet#1
mmmarinho merged 4 commits into
mainfrom
unify-cmake-patterns

Conversation

@mmmarinho

Copy link
Copy Markdown
Contributor

This PR updates submodule pointers and adds a CMake cheatsheet.

Changes

  • Submodule updates — all 14 submodules now point to their latest commits, including:
    • Docker/Compose standardization (COPY instead of volume mounts, uniform build config)
    • Script install unification (install(DIRECTORY scripts/) with FILE_PERMISSIONS)
  • cheatsheet.md — documents all 14 CMake block patterns used across the SAS codebase (shared/static libraries, pybind11 modules, ROS2 binaries, message definitions, Gazebo plugins, third-party subdirectories, install blocks, and more).

- Update submodule pointers to include:
  - Docker/Compose standardization (COPY instead of volume mounts)
  - Script install unification (install(DIRECTORY scripts/))
- Add cheatsheet.md documenting all CMake block patterns used across SAS
@mmmarinho

Copy link
Copy Markdown
Contributor Author

Additional changes pushed to this PR

  • CMake VERSION unification: All 14 submodules now use cmake_minimum_required(VERSION 3.11) (previously most were 3.8). CMake 3.11 is required for pybind11 embedding and block scope support.
  • Cheatsheet updated: cheatsheet.md now reflects VERSION 3.11 as the standard across the stack.

PRs created in submodules

Repository PR Target
SmartArmStack/sas_common #7 (docker-standardize) jazzy
SmartArmStack/sas_conversions #1 (docker-standardize) jazzy
SmartArmStack/sas_datalogger #4 (docker-standardize) jazzy
SmartArmStack/sas_msgs pushed to jazzy jazzy
SmartArmStack/sas_robot_driver #16 (docker-standardize) jazzy
SmartArmStack/sas_robot_kinematics #1 (docker-standardize) jazzy
MarinhoLab/sas_force_sensor #3 (docker-standardize) jazzy
MarinhoLab/sas_force_sensor_bota #1 (docker-standardize) jazzy
MarinhoLab/sas_kuka_control_template #2 main
MarinhoLab/sas_robot_driver_coppeliasim #1 (docker-standardize) jazzy
MarinhoLab/sas_robot_driver_gazebo #1 (docker-standardize) jazzy
MarinhoLab/sas_robot_driver_kuka #1 (docker-standardize) jazzy
MarinhoLab/sas_robot_driver_ur #2 (docker-standardize) jazzy
MarinhoLab/sas_ur_control_template #8 main

Comment posted by an AI agent (OpenHands) on behalf of the user.

@mmmarinho

Copy link
Copy Markdown
Contributor Author

Additional updates

  • Submodule overview table added to cheatsheet.md — lists all 15 packages with their owner, target branch, and which CMake patterns each uses (shared library, pybind11, messages, scripts).
  • All submodules re-updated to their latest remote commits (merge commits for sas_kuka_control_template and sas_ur_control_template now pushed).
  • All 14 repos confirmed pushed — no remaining local-only commits.

Comment posted by an AI agent (OpenHands) on behalf of the user.

@mmmarinho
mmmarinho marked this pull request as ready for review August 15, 2026 20:03
@mmmarinho
mmmarinho merged commit f5fcb4f into main Aug 15, 2026
4 of 5 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