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

FreeBSD: Fails to build; "no viable conversion from 'ByteCursor' (aka 'aws_byte_cursor') to 'All" #3012

Closed
nunotexbsd opened this issue Jun 23, 2024 · 2 comments · Fixed by #3015
Labels
bug This issue is a bug. p2 This is a standard priority issue pending-release This issue will be fixed by an approved PR that hasn't been released yet.

Comments

@nunotexbsd
Copy link

Describe the bug

After upgrading dependencies to latest versions:

aws-c-common: Update to 0.9.23
aws-crt-cpp: Update to 0.27.0
aws-c-cal: Update to 0.7.0

[ 51% 463/879] /usr/bin/c++ -DAWS_AUTH_USE_IMPORT_EXPORT -DAWS_CAL_USE_IMPORT_EXPORT -DAWS_CHECKSUMS_USE_IMPORT_EXPORT -DAWS_COMMON_USE_IMPORT_EXPORT -DAWS_COMPRESSION_USE_IMPORT_EXPORT -DAWS_CRT_CPP_USE_IMPORT_E
XPORT -DAWS_EVENT_STREAM_USE_IMPORT_EXPORT -DAWS_HAS_ALIGNED_ALLOC -DAWS_HTTP_USE_IMPORT_EXPORT -DAWS_IO_USE_IMPORT_EXPORT -DAWS_MQTT_USE_IMPORT_EXPORT -DAWS_S3_USE_IMPORT_EXPORT -DAWS_SDKUTILS_USE_IMPORT_EXPORT
-DAWS_SDK_VERSION_MAJOR=1 -DAWS_SDK_VERSION_MINOR=11 -DAWS_SDK_VERSION_PATCH=353 -DAWS_TEST_REGION=US_EAST_1 -DAWS_USE_KQUEUE -DCURL_HAS_H2 -DCURL_HAS_TLS_PROXY -DENABLED_REQUEST_COMPRESSION -DENABLED_ZLIB_REQUES
T_COMPRESSION -DENABLE_CURL_CLIENT -DENABLE_CURL_LOGGING -DENFORCE_TLS_V1_2 -DHAS_PATHCONF -DHAS_UMASK -DPLATFORM_LINUX -DSMITHY_EXPORTS=1 -DUSE_IMPORT_EXPORT=1 -Daws_cpp_sdk_core_EXPORTS -I/wrkdirs/usr/ports/dev
el/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/include/aws/core/platform/refs -I/wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/include -O2 -pipe -fstack-protect
or-strong -isystem /usr/local/include -fno-strict-aliasing  -isystem /usr/local/include -O2 -pipe -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing  -isystem /usr/local/include  -DNDEBUG -
fPIC -fno-exceptions -std=c++11 -Wall -pedantic -Wextra -fPIC -MD -MT src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/source/utils/crypto/factory/Factories.cpp.o -MF src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-s
dk-core.dir/source/utils/crypto/factory/Factories.cpp.o.d -o src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/source/utils/crypto/factory/Factories.cpp.o -c /wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-c
pp-1.11.353/src/aws-cpp-sdk-core/source/utils/crypto/factory/Factories.cpp
FAILED: src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/source/utils/crypto/factory/Factories.cpp.o
/usr/bin/c++ -DAWS_AUTH_USE_IMPORT_EXPORT -DAWS_CAL_USE_IMPORT_EXPORT -DAWS_CHECKSUMS_USE_IMPORT_EXPORT -DAWS_COMMON_USE_IMPORT_EXPORT -DAWS_COMPRESSION_USE_IMPORT_EXPORT -DAWS_CRT_CPP_USE_IMPORT_EXPORT -DAWS_EVE
NT_STREAM_USE_IMPORT_EXPORT -DAWS_HAS_ALIGNED_ALLOC -DAWS_HTTP_USE_IMPORT_EXPORT -DAWS_IO_USE_IMPORT_EXPORT -DAWS_MQTT_USE_IMPORT_EXPORT -DAWS_S3_USE_IMPORT_EXPORT -DAWS_SDKUTILS_USE_IMPORT_EXPORT -DAWS_SDK_VERSI
ON_MAJOR=1 -DAWS_SDK_VERSION_MINOR=11 -DAWS_SDK_VERSION_PATCH=353 -DAWS_TEST_REGION=US_EAST_1 -DAWS_USE_KQUEUE -DCURL_HAS_H2 -DCURL_HAS_TLS_PROXY -DENABLED_REQUEST_COMPRESSION -DENABLED_ZLIB_REQUEST_COMPRESSION -
DENABLE_CURL_CLIENT -DENABLE_CURL_LOGGING -DENFORCE_TLS_V1_2 -DHAS_PATHCONF -DHAS_UMASK -DPLATFORM_LINUX -DSMITHY_EXPORTS=1 -DUSE_IMPORT_EXPORT=1 -Daws_cpp_sdk_core_EXPORTS -I/wrkdirs/usr/ports/devel/aws-sdk-cpp/
work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/include/aws/core/platform/refs -I/wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/include -O2 -pipe -fstack-protector-strong -isys
tem /usr/local/include -fno-strict-aliasing  -isystem /usr/local/include -O2 -pipe -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing  -isystem /usr/local/include  -DNDEBUG -fPIC -fno-excep
tions -std=c++11 -Wall -pedantic -Wextra -fPIC -MD -MT src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/source/utils/crypto/factory/Factories.cpp.o -MF src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/sou
rce/utils/crypto/factory/Factories.cpp.o.d -o src/aws-cpp-sdk-core/CMakeFiles/aws-cpp-sdk-core.dir/source/utils/crypto/factory/Factories.cpp.o -c /wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src
/aws-cpp-sdk-core/source/utils/crypto/factory/Factories.cpp
/wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/source/utils/crypto/factory/Factories.cpp:379:229: error: no viable conversion from 'ByteCursor' (aka 'aws_byte_cursor') to 'All
ocator *' (aka 'aws_allocator *')
  379 |             return Aws::MakeShared<CRTSymmetricCipher>(s_allocationTag, Aws::Crt::Crypto::SymmetricCipher::CreateAES_256_GCM_Cipher(keyCur, Aws::Crt::Optional<Aws::Crt::ByteCursor>(), Aws::Crt::Optional<A
ws::Crt::ByteCursor>(), aadCur));
      |
                        ^~~~~~
/usr/local/include/aws/crt/crypto/SymmetricCipher.h:64:32: note: passing argument to parameter 'allocator' here
   64 |                     Allocator *allocator = ApiAllocator()) noexcept;
      |                                ^
/wrkdirs/usr/ports/devel/aws-sdk-cpp/work/aws-sdk-cpp-1.11.353/src/aws-cpp-sdk-core/source/utils/crypto/factory/Factories.cpp:401:152: error: no viable conversion from 'Aws::Crt::Optional<Aws::Crt::ByteCursor>' (
aka 'Optional<aws_byte_cursor>') to 'Allocator *' (aka 'aws_allocator *')
  401 |         return Aws::MakeShared<CRTSymmetricCipher>(s_allocationTag, Aws::Crt::Crypto::SymmetricCipher::CreateAES_256_GCM_Cipher(keyCur, ivCur, tagCur, aadCur));
      |                                                                                                                                                        ^~~~~~
/usr/local/include/aws/crt/crypto/SymmetricCipher.h:64:32: note: passing argument to parameter 'allocator' here
   64 |                     Allocator *allocator = ApiAllocator()) noexcept;
      |                                ^
2 errors generated.

Expected Behavior

Compile OK

Current Behavior

Fails to compile

Reproduction Steps

FreeBSD Ports Collection: cd /usr/ports/devel/aws-sdk-cpp && make

Possible Solution

No response

Additional Information/Context

No response

AWS CPP SDK version used

1.11.353

Compiler and Version used

Clang 18.1.6

Operating System and version

FreeBSD 15

@nunotexbsd nunotexbsd added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jun 23, 2024
@DmitriyMusatkin
Copy link
Contributor

DmitriyMusatkin commented Jun 24, 2024

Crt had to make a minor backwards incompat change in the latest rev in 0.27.0 (since the bump from 0.26.x). cpp should pick it up early this week and resolve the issue.

freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this issue Jun 24, 2024
Due to incompatibility changes in devel/aws-crt-cpp (0.27.0), this port is
waiting for upstream fix.

See also: aws/aws-sdk-cpp#3012
@sbiscigl sbiscigl mentioned this issue Jun 24, 2024
11 tasks
@jmklix jmklix added pending-release This issue will be fixed by an approved PR that hasn't been released yet. p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Jun 26, 2024
@jmklix jmklix linked a pull request Jun 26, 2024 that will close this issue
11 tasks
Copy link

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.

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 pending-release This issue will be fixed by an approved PR that hasn't been released yet.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants