Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMAKE issue with latest CRT v0.30.1 #776

Closed
thomas-roos opened this issue Feb 5, 2025 · 3 comments
Closed

CMAKE issue with latest CRT v0.30.1 #776

thomas-roos opened this issue Feb 5, 2025 · 3 comments
Labels
bug This issue is a bug.

Comments

@thomas-roos
Copy link
Contributor

Describe the bug

Due to a change in CMakefiles in CRT
Similar changes like here: aws/aws-sdk-cpp@e932b19#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20a

Needs to applied

Same issue, same solution: aws/aws-sdk-cpp#3281

Expected Behavior

compiles

Current Behavior

cmake breaks

Reproduction Steps

build with latest CRT

Possible Solution

Index: git/CMakeLists.txt
===================================================================
--- git.orig/CMakeLists.txt
+++ git/CMakeLists.txt
@@ -37,10 +37,7 @@ if (${CMAKE_INSTALL_LIBDIR} STREQUAL "li
 endif()
 
 # This is required in order to append /lib/cmake to each element in CMAKE_PREFIX_PATH
-set(AWS_MODULE_DIR "/${CMAKE_INSTALL_LIBDIR}/cmake")
-string(REPLACE ";" "${AWS_MODULE_DIR};" AWS_MODULE_PATH "${CMAKE_PREFIX_PATH}${AWS_MODULE_DIR}")
-# Append that generated list to the module search path
-list(APPEND CMAKE_MODULE_PATH ${AWS_MODULE_PATH})
+find_package(aws-c-common REQUIRED)
 
 if (NOT CMAKE_BUILD_TYPE)
     if (NOT WIN32)

Additional Information/Context

No response

SDK version used

1.34

Environment details (OS name and version, etc.)

Yocto / meta-aws

@thomas-roos thomas-roos added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Feb 5, 2025
thomas-roos added a commit to thomas-roos/meta-aws that referenced this issue Feb 5, 2025
thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
github-actions bot pushed a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
github-actions bot pushed a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
github-actions bot pushed a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Feb 5, 2025
Breaking change in CMAKE file of CRT libs.
aws/aws-iot-device-sdk-cpp-v2#776

(cherry picked from commit 22d2bed)
@sfod sfod added pending-release This issue will be fixed by an approved PR that hasn't been released yet. and removed needs-triage This issue or PR still needs to be triaged. labels Feb 5, 2025
@sfod
Copy link
Contributor

sfod commented Feb 5, 2025

#769 should fix the issue. As soon as CI succeeds, we'll merge it and make a new release

@sfod
Copy link
Contributor

sfod commented Feb 6, 2025

This issue is fixed in v1.35.0 release

@sfod sfod closed this as completed Feb 6, 2025
Copy link

github-actions bot commented Feb 6, 2025

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

@sfod sfod removed the pending-release This issue will be fixed by an approved PR that hasn't been released yet. label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
Development

No branches or pull requests

2 participants