Building grpc on VS 2017

222 views
Skip to first unread message

whe...@gmail.com

unread,
Jul 10, 2018, 2:36:15 PM7/10/18
to grpc.io
Attempting to follow the non-ninja instructions for building on Windows here: https://github.com/grpc/grpc/blob/master/BUILDING.md
But I've replaced
-G "Visual Studio 14 2015"

with
-G "Visual Studio 15 2017"

I ignore a lot of seemingly-innocuous warnings (signed mismatch, zero-len array, /QSpectre etc)

But getting tons of errors like

"F:\grpc\.build\ALL_BUILD.vcxproj" (default target) (1) ->
"F:\grpc\.build\third_party\boringssl\crypto\asn1\asn1.vcxproj" (default target) (8) ->
(ClCompile target) ->
  f:\grpc\third_party\boringssl\crypto\asn1\f_string.c(82): error C2220: warning treated as error - no 'object' file generated [F:\grpc\.build\third_party\bor
ingssl\crypto\asn1\asn1.vcxproj]
  f:\grpc\third_party\boringssl\crypto\asn1\f_int.c(88): error C2220: warning treated as error - no 'object' file generated [F:\grpc\.build\third_party\boring
ssl\crypto\asn1\asn1.vcxproj]
  f:\grpc\third_party\boringssl\crypto\asn1\f_enum.c(84): error C2220: warning treated as error - no 'object' file generated [F:\grpc\.build\third_party\borin
gssl\crypto\asn1\asn1.vcxproj]

Any suggestions?  Is this VS 2017 build supported?

Thanks
john

Jan Tattermusch

unread,
Jul 13, 2018, 8:12:46 AM7/13/18
to grpc.io

The "error C2220: warning treated as error" basically means that the warning actually didn't get ignored - so perhaps you are setting the options incorrectly when you say "I ignore a lot of warnings" - You don't specify which options you are using to ignore the warning so I can't help you there.

gRPC does treat warnings as errors by default, but we have ignore flags set in our CMakeLists.txt to be able to compile with VS2017 sucessfully.

VS2017 certainly is supported and we have a continously running tests to check that (and the test is passing, e.g. https://source.cloud.google.com/results/invocations/b2381313-9d90-4aaa-b39c-138d5138428c/targets/github%2Fgrpc%2Fcpp_windows_dbg_native_x64_cmake_vs2017_buildonly/tests).

What's your VisualStudio version? There might be some new warnings added in the freshest version of VS2017 (the QSpectre you're mentioning might be this one https://github.com/grpc/grpc/issues/15647).

John Lilley

unread,
Jul 13, 2018, 3:16:33 PM7/13/18
to jtatte...@google.com, grp...@googlegroups.com
Jan,
Thanks for the follow-up.  We've decided not to use gRPC because it doesn't really help with rabbitMQ-protobuf integration.
John Lilley

--
You received this message because you are subscribed to a topic in the Google Groups "grpc.io" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/grpc-io/mftj4FraxV0/unsubscribe.
To unsubscribe from this group and all its topics, send an email to grpc-io+u...@googlegroups.com.
To post to this group, send email to grp...@googlegroups.com.
Visit this group at https://groups.google.com/group/grpc-io.
To view this discussion on the web visit https://groups.google.com/d/msgid/grpc-io/169e0b22-0542-424d-ba31-87f7294aeb88%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Reply all
Reply to author
Forward
0 new messages