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

Unable to compile AWS SDK with Visual Studio 2017 #2656

Open
ayushgupta-aic opened this issue Sep 4, 2023 · 14 comments
Open

Unable to compile AWS SDK with Visual Studio 2017 #2656

ayushgupta-aic opened this issue Sep 4, 2023 · 14 comments
Labels
bug This issue is a bug. build-problem problems with building this sdk closing-soon This issue will automatically close in 4 days unless further comments are made. p3 This is a minor priority issue response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days.

Comments

@ayushgupta-aic
Copy link

Describe the bug

Unable to compile the v1.11 (or even the main branch) CPP SDK source code using VS 2017 and Windows SDK 10.0.22000.0

Expected Behavior

Expect to compile fine

Current Behavior

C:\Program Files (x86)\Windows Kits\10\Include\10.0.22000.0\um\rpcnsi.h(368): error C2872: 'UUID': ambiguous symbol (compiling source file C:\work\awscppsdk\aws-sdk-cpp-build\src\aws-cpp-sdk-core\ub_core.cpp)
13>C:\Program Files (x86)\Windows Kits\10\Include\10.0.22000.0\shared\rpcdce.h(83): note: could be 'GUID UUID' (compiling source file C:\work\awscppsdk\aws-sdk-cpp-build\src\aws-cpp-sdk-core\ub_core.cpp)
13>C:\work\awscppsdk\aws-sdk-cpp\src\aws-cpp-sdk-core\include\aws/core/utils/UUID.h(21): note: or 'Aws::Utils::UUID' (compiling source file C:\work\awscppsdk\aws-sdk-cpp-build\src\aws-cpp-sdk-core\ub_core.cpp)

Reproduction Steps

Build commands:
cmake -G "Visual Studio 15 2017 Win64" c:\work\awscppsdk\aws-sdk-cpp -DCMAKE_BUILD_TYPE=Release -DBUILD_ONLY="core;s3;glacier;sqs;sns" -DBUILD_SHARED_LIBS=FALSE -DFORCE_SHARED_CRT=FALSE -DENABLE_TESTING=FALSE
followed by compilation of the projects in Visual Studio 2017

Possible Solution

No response

Additional Information/Context

No response

AWS CPP SDK version used

1.11

Compiler and Version used

Visual Studio 2017/ Windows SDK 10.0.22000.0

Operating System and version

Windows 11

@ayushgupta-aic ayushgupta-aic added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Sep 4, 2023
@yasminetalby yasminetalby self-assigned this Sep 5, 2023
@yasminetalby yasminetalby added p2 This is a standard priority issue investigating This issue is being investigated and/or work is in progress to resolve the issue. and removed needs-triage This issue or PR still needs to be triaged. labels Sep 5, 2023
@yasminetalby
Copy link
Contributor

Hello @ayushgupta-aic ,

Thank you very much for your submission.
It seems that their might be an issue with how you are qualifying UUID in your rpcnsi.h file.
C:\Program Files (x86)\Windows Kits\10\Include\10.0.22000.0\um\rpcnsi.h(368): error C2872: 'UUID': ambiguous symbol

If you intend to use Aws::Utils::UUID need help with this, could you please provide a snippet of your code and use case of UUID in rpcnsi.f ?

Best,

Yasmine

@yasminetalby yasminetalby added guidance Question that needs advice or information. response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. p3 This is a minor priority issue and removed p2 This is a standard priority issue bug This issue is a bug. investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Sep 5, 2023
@ayushgupta-aic
Copy link
Author

ayushgupta-aic commented Sep 5, 2023

@yasminetalby : Thanks for your response.
We are just trying to compile the AWS CPP codebase. It does not include any of our custom code. rpcnsi.h is part of Windows SDK.

More specifically, while trying to compile the aws-cpp-sdk-core project.
Attached if the full compilation log
aws-cpp-sdk-core.log

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Sep 6, 2023
@yasminetalby
Copy link
Contributor

Hello @ayushgupta-aic ,

Thank you very much for your response and for providing the logs. It seems that there is a conflict issue with the Windows SDK definition of UUID typedef GUID UUID Working on a fix right now!

Best,

Yasmine

@yasminetalby yasminetalby removed the guidance Question that needs advice or information. label Sep 6, 2023
@yasminetalby yasminetalby mentioned this issue Sep 6, 2023
11 tasks
@yasminetalby yasminetalby added the pending-release This issue will be fixed by an approved PR that hasn't been released yet. label Sep 12, 2023
@yasminetalby
Copy link
Contributor

Hello @ayushgupta-aic ,

This issue should be fixed in v1.11.162 of the SDK.
Let me know if this resolves your issue.

Thank you very much for your collaboration.

Best regards,

Yasmine

@yasminetalby yasminetalby added response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. and removed pending-release This issue will be fixed by an approved PR that hasn't been released yet. labels Sep 13, 2023
@ayushgupta-aic
Copy link
Author

hi @yasminetalby . I am still encountering the same errors. Tried with both v1.11.162 and v1.11.163

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Sep 15, 2023
@ayushgupta-aic
Copy link
Author

@yasminetalby : Any update on this?

@jmklix
Copy link
Member

jmklix commented Mar 13, 2024

Are you still running into this with the latest version of the sdk? I've tried reproducing this, but I can't seem to reproduce the same error that you are seeing.

@jmklix jmklix added bug This issue is a bug. response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. labels Mar 13, 2024
@ayushgupta-aic
Copy link
Author

ayushgupta-aic commented Mar 14, 2024 via email

@jmklix jmklix removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Mar 14, 2024
@jmklix
Copy link
Member

jmklix commented Mar 14, 2024

Using the latest version is always our recommendation. Currently that is 1.11.285

@jmklix jmklix added the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Mar 14, 2024
@ayushgupta-aic
Copy link
Author

ayushgupta-aic commented Mar 19, 2024 via email

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Mar 20, 2024
@jmklix
Copy link
Member

jmklix commented Mar 22, 2024

Can you add -DAWS_STATIC_MSVC_RUNTIME_LIBRARY=ON to your cmake config. Also just as a reminder please make sure you're using the same MSVC runtime version.

I'm able to successfully build on windows with the following:

cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX="<install-path>" -DBUILD_ONLY="s3;glacier;sqs;sns;core" -DFORCE_SHARED_CRT=OFF -DBUILD_SHARED_LIBS=OFF -DENABLE_TESTING=OFF -DAWS_STATIC_MSVC_RUNTIME_LIBRARY=ON

Can you make sure your build and install directories are clean while doing this?

@jmklix jmklix added the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Mar 22, 2024
@ayushgupta-aic
Copy link
Author

ayushgupta-aic commented Apr 1, 2024 via email

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label Apr 2, 2024
@jmklix jmklix added the build-problem problems with building this sdk label Apr 5, 2024
@jmklix
Copy link
Member

jmklix commented May 24, 2024

Can you take a look at this guide I'm working for how to build this sdk in Visual Studio. Please let me know if you have any questions and/or if you still can't get the sdk to build in Visual Studio 2017

@jmklix jmklix added the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days. label May 24, 2024
Copy link

github-actions bot commented Jun 4, 2024

Greetings! It looks like this issue hasn’t been active in longer than a week. We encourage you to check if this is still an issue in the latest release. Because it has been longer than a week since the last update on this, and in the absence of more information, we will be closing this issue soon. If you find that this is still a problem, please feel free to provide a comment or add an upvote to prevent automatic closure, or if the issue is already closed, please feel free to open a new one.

@github-actions github-actions bot added the closing-soon This issue will automatically close in 4 days unless further comments are made. label Jun 4, 2024
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. build-problem problems with building this sdk closing-soon This issue will automatically close in 4 days unless further comments are made. p3 This is a minor priority issue response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 2 days.
Projects
None yet
Development

No branches or pull requests

3 participants