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

CI failed when building azure-core #4131

Closed
Jinming-Hu opened this issue Nov 28, 2022 · 3 comments · Fixed by #4132
Closed

CI failed when building azure-core #4131

Jinming-Hu opened this issue Nov 28, 2022 · 3 comments · Fixed by #4132
Assignees
Labels
Azure.Core bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library.

Comments

@Jinming-Hu
Copy link
Member

         base64.cpp
         context.cpp
         md5.cpp
         sha_hash.cpp
         datetime.cpp
         environment.cpp
         environment_log_level_listener.cpp
         etag.cpp
         exception.cpp
         bearer_token_authentication_policy.cpp
         http.cpp
         http_sanitizer.cpp
         log_policy.cpp
         policy.cpp
         raw_response.cpp
         request.cpp
         request_activity_policy.cpp
         retry_policy.cpp
         Running Code Analysis for C/C++...
     3>D:\a\_work\1\s\build\vcpkg_installed\x86-windows-static\include\wil\result_macros.h(1324,9): error C2220: the following warning is treated as an error [D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj]
     3>D:\a\_work\1\s\build\vcpkg_installed\x86-windows-static\include\wil\result_macros.h(1314): warning C6387: 'Temp_value_#2184' could be '0':  this does not adhere to the specification for the function 'GetProcAddress'. : Lines: 1302, 1303, 1305, 1306, 1308, 1312, 1314 [D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj]
         Generating Code...
         Compiling...
         telemetry_policy.cpp
         transport_policy.cpp
         url.cpp
         user_agent.cpp
         body_stream.cpp
         random_access_file_body_stream.cpp
         logger.cpp
         operation_status.cpp
         strings.cpp
         tracing.cpp
         uuid.cpp
         Running Code Analysis for C/C++...
         Generating Code...
     3>Done Building Project "D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj" (default targets) -- FAILED.
     1>Done Building Project "D:\a\_work\1\s\build\sdk\storage\azure-storage-common\azure-storage-common.vcxproj" (default targets) -- FAILED.

Build FAILED.

       "D:\a\_work\1\s\build\sdk\storage\azure-storage-common\azure-storage-common.vcxproj" (default target) (1) ->
       "D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj" (default target) (3) ->
       (ClCompile target) -> 
         D:\a\_work\1\s\build\vcpkg_installed\x86-windows-static\include\wil\result_macros.h(1314): warning C6387: 'Temp_value_#2184' could be '0':  this does not adhere to the specification for the function 'GetProcAddress'. : Lines: 1302, 1303, 1305, 1306, 1308, 1312, 1314 [D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj]


       "D:\a\_work\1\s\build\sdk\storage\azure-storage-common\azure-storage-common.vcxproj" (default target) (1) ->
       "D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj" (default target) (3) ->
       (ClCompile target) -> 
         D:\a\_work\1\s\build\vcpkg_installed\x86-windows-static\include\wil\result_macros.h(1324,9): error C2220: the following warning is treated as an error [D:\a\_work\1\s\build\sdk\core\azure-core\azure-core.vcxproj]

    1 Warning(s)
    1 Error(s)

Time Elapsed 00:01:13.71
@Jinming-Hu Jinming-Hu added bug This issue requires a change to an existing behavior in the product in order to be resolved. Azure.Core Client This issue points to a problem in the data-plane of the library. labels Nov 28, 2022
@Jinming-Hu
Copy link
Member Author

@RickWinter RickWinter assigned ahsonkhan and unassigned RickWinter Nov 28, 2022
@RickWinter
Copy link
Member

@ahsonkhan, @LarryOsterman This looks to be the same warning issue discussed last week. Is there a fix available?

@ahsonkhan
Copy link
Member

Yep. Fixed in:
#4132

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Azure.Core bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library.
Projects
None yet
3 participants