Skip to content

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Apr 25, 2024

This PR #137 introduces a regresion. CMP0148 is new in CMake 3.27. The main CMakeLists.txt is using 3.5, I updated the cmake/shiboken_helper.cmake version to 3.5 too.

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from sloretz as a code owner April 25, 2024 07:59
@ahcorde ahcorde requested a review from clalancette April 25, 2024 07:59
@ahcorde ahcorde self-assigned this Apr 25, 2024
@ahcorde
Copy link
Contributor Author

ahcorde commented Apr 25, 2024

  • Linux Build Status

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@ahcorde
Copy link
Contributor Author

ahcorde commented Apr 25, 2024

The CMake version installed in rhel is:

10:03:04  cmake                                x86_64  3.20.2-9.el9_3           appstream  6.7 M

Trying a different approach

  • Linux Build Status

@ahcorde
Copy link
Contributor Author

ahcorde commented Apr 25, 2024

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor

(and let's remember to backport this to Jazzy as well)

@clalancette clalancette merged commit 4fe5bce into rolling Apr 25, 2024
@clalancette clalancette deleted the ahcorde/rolling/cmake_shiboken branch April 25, 2024 12:25
@clalancette
Copy link
Contributor

@Mergifyio backport jazzy

Copy link

mergify bot commented Apr 25, 2024

backport jazzy

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Apr 25, 2024
* Older CMake doesn't have the policy, so skip it there.

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 4fe5bce)

# Conflicts:
#	cmake/shiboken_helper.cmake
clalancette pushed a commit that referenced this pull request Apr 25, 2024
* Older CMake doesn't have the policy, so skip it there.

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
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