Skip to content
Snippets Groups Projects

Use new CMAKE_ARGS variable introduced in industrial_ci#369

Merged Prada Sarasola, Miguel requested to merge use-cmake-args into master
All threads resolved!

This is not really WIP, since the intended change is already there, but it depends on industrial_ci#369 being merged.

There, in preparation for the migration to the new colcon&ROS2-enabled refactor of industrial_ci, the new CMAKE_ARGS variable is introduced. It allows passing the required arguments in a way that is not specific to catkin-tools.

Edited by Prada Sarasola, Miguel

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ping, can we merge this?

    I want to start running tests against industrial_ci#361, which should become the master version relatively soon.

  • added 16 commits

    Compare with previous version

  • Prada Sarasola, Miguel resolved all discussions

    resolved all discussions

  • added 1 commit

    • 701c80c4 - Remove quotes from YAML strings

    Compare with previous version

  • Do you have any test jobs using this branch to validate that nothing strange happens?

  • Yes, I tested it on forks1 of flexbotics_legacy, manipulation and vision_pipelines, and nothing seems to break.


    [1] In order not to disturb with test branches. That's not great for visibility, I know...

  • merged

  • Jon Azpiazu mentioned in commit b9934219

    mentioned in commit b9934219

  • Jon Azpiazu mentioned in merge request !31 (merged)

    mentioned in merge request !31 (merged)

  • Please register or sign in to reply
    Loading