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

Doesn't compile with latest aws-* dependencies with CMAKE updates #3281

Open
1 task
nunotexbsd opened this issue Jan 30, 2025 · 5 comments
Open
1 task

Doesn't compile with latest aws-* dependencies with CMAKE updates #3281

nunotexbsd opened this issue Jan 30, 2025 · 5 comments
Labels
bug This issue is a bug. needs-review This issue or pull request needs review from a core team member. p1 This is a high priority issue

Comments

@nunotexbsd
Copy link

nunotexbsd commented Jan 30, 2025

Describe the bug

Hello,

Recently all dependencies of aws-sdk-cpp received cmake changes, like this one: awslabs/aws-c-cal#203
being the last change in: awslabs/aws-crt-cpp#702

Latest release fails with details bellow.

Will a new release be created with CMake changes too?
See also: awslabs/aws-c-cal#204 (comment)

CMake Error at CMakeLists.txt:247 (include):
  include could not find requested file:

    AwsFindPackage


CMake Error at CMakeLists.txt:250 (aws_use_package):
  Unknown CMake command "aws_use_package".


-- Configuring incomplete, errors occurred!

Regression Issue

  • Select this option if this issue appears to be a regression.

Expected Behavior

Build with recently cmake updated deps

Current Behavior

doesn't build

Reproduction Steps

FreeBSD framework:

After all deps updated (not committed yet):

cd /usr/ports/devel/aws-sdk-cpp && make

Possible Solution

Update cmake modules and config to match deps updates

Additional Information/Context

No response

AWS CPP SDK version used

1.11.495

Compiler and Version used

Clang 19.1.7

Operating System and version

FreeBSD 15 amd64

@nunotexbsd nunotexbsd added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jan 30, 2025
@thomas-roos
Copy link

Also affected by this with meta-aws Yocto builds.
Like here: https://github.com/aws4embeddedlinux/meta-aws/actions/runs/13065076486/job/36455932376?pr=11366

@nunotexbsd
Copy link
Author

@thomas-roos

I have all aws-* in queue waiting for aws-sdk-cpp to get fixed.
I believe that a fix could be done easily by changing cmake to look to new location on deps:

For example, all deps have changed this way:

 include/aws/common/zero.h
 include/aws/common/zero.inl
 include/aws/testing/aws_test_harness.h
-lib/aws-c-common/cmake/aws-c-common-config.cmake
-lib/aws-c-common/cmake/shared/aws-c-common-targets-%%CMAKE_BUILD_TYPE%%.cmake
-lib/aws-c-common/cmake/shared/aws-c-common-targets.cmake
-lib/cmake/AwsCFlags.cmake
-lib/cmake/AwsCRuntime.cmake
-lib/cmake/AwsCheckHeaders.cmake
-lib/cmake/AwsFeatureTests.cmake
-lib/cmake/AwsFindPackage.cmake
-lib/cmake/AwsLibFuzzer.cmake
-lib/cmake/AwsSIMD.cmake
-lib/cmake/AwsSanitizers.cmake
-lib/cmake/AwsSharedLibSetup.cmake
-lib/cmake/AwsTestHarness.cmake
+lib/cmake/aws-c-common/aws-c-common-config.cmake
+lib/cmake/aws-c-common/modules/AwsCFlags.cmake
+lib/cmake/aws-c-common/modules/AwsCRuntime.cmake
+lib/cmake/aws-c-common/modules/AwsCheckHeaders.cmake
+lib/cmake/aws-c-common/modules/AwsFeatureTests.cmake
+lib/cmake/aws-c-common/modules/AwsFindPackage.cmake
+lib/cmake/aws-c-common/modules/AwsLibFuzzer.cmake
+lib/cmake/aws-c-common/modules/AwsSIMD.cmake
+lib/cmake/aws-c-common/modules/AwsSanitizers.cmake
+lib/cmake/aws-c-common/modules/AwsSharedLibSetup.cmake
+lib/cmake/aws-c-common/modules/AwsTestHarness.cmake
+lib/cmake/aws-c-common/shared/aws-c-common-targets-%%CMAKE_BUILD_TYPE%%.cmake
+lib/cmake/aws-c-common/shared/aws-c-common-targets.cmake
 lib/libaws-c-common.so
 lib/libaws-c-common.so.1
 lib/libaws-c-common.so.1.0.0

@nunotexbsd
Copy link
Author

@nunotexbsd
Copy link
Author

nunotexbsd commented Jan 31, 2025

@thomas-roos

I have it building right now by adding args to cmake:

-DCMAKE_MODULE_PATH=${LOCALBASE}/lib/cmake/aws-c-common/modules

$LOCALBASE is /usr/local by default at FreeBSD

@sbiscigl
Copy link
Contributor

yeah we're working on a CRT update, note that the supported version of the aws-c-* dependencies are the ones that we are tracking in submodules and that they did do a minor version bump this time to indicate a breaking change

we are actively looking at it with high priority right now though

I believe that a fix could be done easily by changing cmake to look to new location on deps:

this will likely be the fix, will update when i have it out

@sbiscigl sbiscigl added p1 This is a high priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Jan 31, 2025
@sbiscigl sbiscigl mentioned this issue Jan 31, 2025
11 tasks
@sbiscigl sbiscigl added the needs-review This issue or pull request needs review from a core team member. label Jan 31, 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. needs-review This issue or pull request needs review from a core team member. p1 This is a high priority issue
Projects
None yet
Development

No branches or pull requests

3 participants