Skip to content

Conversation

@richiware
Copy link
Member

Increase version to one is being testing our CI

@richiware richiware changed the title Increase cmake minimum required Increase cmake minimum required [16436] Dec 7, 2022
@richiware richiware changed the base branch from master to develop December 7, 2022 13:03
Signed-off-by: Ricardo González Moreno <[email protected]>
@richiware richiware force-pushed the feature/increase-cmake-minimum branch from 7ecedfb to 9dfcab3 Compare December 7, 2022 13:04
Signed-off-by: Ricardo González Moreno <[email protected]>
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

Signed-off-by: Pablo Garrido <[email protected]>
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@pablogs9 pablogs9 merged commit beb454b into develop Dec 8, 2022
@pablogs9 pablogs9 deleted the feature/increase-cmake-minimum branch December 8, 2022 09:32
###############################################################################
# CMake build rules for Micro CDR
###############################################################################
cmake_minimum_required(VERSION 3.5.0 FATAL_ERROR)
Copy link

@Ryanf55 Ryanf55 Feb 27, 2023

Choose a reason for hiding this comment

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

CI environment has not yet been updated to have this new version. Linux jobs are failing due to it.
http://jenkins.eprosima.com:8080/view/Micro%20XRCE/job/Micro-CDR%20Nightly%20Master%20Linux/1232/console

[Micro-CDR Nightly Master Linux] $ /home/jenkins/tools/hudson.plugins.cmake.CmakeTool/3.10.0/bin/cmake --version
cmake version 3.10.0
...
CMake Error at CMakeLists.txt:18 (cmake_minimum_required):
  CMake 3.15 or higher is required.  You are running version 3.10.0


-- Configuring incomplete, errors occurred!

Copy link
Contributor

Choose a reason for hiding this comment

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

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.

5 participants