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 Error at CMakeLists.txt:317 (aws_add_sanitizers): Unknown CMake command "aws_add_sanitizers" #548

Closed
nunotexbsd opened this issue Oct 4, 2023 · 6 comments
Assignees
Labels
bug This issue is a bug. p2 This is a standard priority issue

Comments

@nunotexbsd
Copy link

nunotexbsd commented Oct 4, 2023

Describe the bug

Configure fails:

===>  Configuring for aws-crt-cpp-0.24.0
===>  Performing out-of-source build
/bin/mkdir -p /wrkdirs/usr/ports/devel/aws-crt-cpp/work/.build
-- AWS CRT C++ 0.24.0
-- The CXX compiler identification is Clang 14.0.5
-- The C compiler identification is Clang 14.0.5
<snip>
-- aws-c-cal found target: AWS::crypto
-- crypto Include Dir: /usr/include
-- aws-c-cal found target: AWS::crypto
-- crypto Include Dir: /usr/include
-- S2N found target: AWS::crypto
-- crypto Include Dir: /usr/include
-- S2N found target: AWS::crypto
-- crypto Include Dir: /usr/include
-- aws-c-cal found target: AWS::crypto
-- crypto Include Dir: /usr/include
CMake Error at CMakeLists.txt:317 (aws_add_sanitizers):
  Unknown CMake command "aws_add_sanitizers".


-- Configuring incomplete, errors occurred!
*** Error code 1

Configure fails:

Expected Behavior

configure ok

Current Behavior

configure fails related to CMAkeLists: 317

aws_add_sanitizers(${PROJECT_NAME})

Reproduction Steps

cmake

Possible Solution

No response

Additional Information/Context

No response

aws-crt-cpp version used

0.24.0

Compiler and version used

clang 14.0.5

Operating System and version

FreeBSD 13.2 amd64

@nunotexbsd nunotexbsd added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Oct 4, 2023
@yasminetalby
Copy link
Contributor

Hello @nunotexbsd ,

Could you please provide a little more details on your use case? Specifically the reproduction steps. How did you install and build the SDK?

Did you perform a recursive clone git clone --recursiveor initialize the submodules via git submodule update --init?

Best regards,

Yasmine

@yasminetalby yasminetalby self-assigned this Oct 4, 2023
@yasminetalby yasminetalby added response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 7 days. p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Oct 4, 2023
@nunotexbsd
Copy link
Author

Hello @yasminetalby ,

Build is doing on FreeBSD ports framework (aws-crt-cpp port)and it is a trivial update from 2.23.1 to 2.24.0:
https://people.freebsd.org/~eduardo/logs/aws-crt-cpp/aws-crt-cpp-0.24.0.log

It seems related to 7c15c5b since previous builds are ok.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 7 days. label Oct 4, 2023
@yasminetalby yasminetalby added the investigating This issue is being investigated and/or work is in progress to resolve the issue. label Oct 4, 2023
@nunotexbsd
Copy link
Author

@yasminetalby

Comment aws_add_sanitizers fixes build:

--- CMakeLists.txt.orig 2023-10-04 19:28:22 UTC
+++ CMakeLists.txt
@@ -314,7 +314,7 @@ aws_use_package(aws-c-s3)
 aws_use_package(aws-c-event-stream)
 aws_use_package(aws-c-s3)

-aws_add_sanitizers(${PROJECT_NAME})
+#aws_add_sanitizers(${PROJECT_NAME})

 target_link_libraries(${PROJECT_NAME} PUBLIC ${DEP_AWS_LIBS})

https://people.freebsd.org/~eduardo/logs/aws-crt-cpp/aws-crt-cpp-0.24.0_HOTFIX.log

Hope that helps.
Thanks

@thomas-roos
Copy link

we (https://github.com/aws4embeddedlinux/meta-aws) can also confirm this issue

thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Oct 5, 2023
thomas-roos added a commit to aws4embeddedlinux/meta-aws that referenced this issue Oct 5, 2023
@graebm graebm removed the investigating This issue is being investigated and/or work is in progress to resolve the issue. label Oct 5, 2023
@xiazhvera
Copy link
Contributor

Fixed in latest release: https://github.com/awslabs/aws-crt-cpp/releases/tag/v0.24.2

@nunotexbsd
Copy link
Author

Thanks for fix!

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. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

5 participants