-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
gRPCTargets.cmake have invalid target properties #20578
Comments
When you say "As result it is impossible to build project that use gRPC, because 'libprotoc.lib' not found.", can you include the exact steps you've made an and the corresponding error log? |
|
@hdnn can you reproduce using cmake only, without using conan? (we don't officially conan, so it's not clear to me if this is a problem in our cmake build or in the conan recipe). |
Seems similar to #12424? |
So far, I have not been able to get the Conan recipe to build on my Windows machine to reproduce this. |
Closing as "cannot reproduce". |
What version of gRPC and what language are you using?
1.23.0
What operating system (Linux, Windows,...) and version?
Windows 10 Pro
What runtime / compiler are you using (e.g. python version or version of gcc)
MSBuilder
What did you do?
I compile gRPC, with generating gRPCTargets.cmake file(all dependences is "package")
What did you expect to see?
In gRPCTargets.cmake:
_set_target_properties(gRPC::grpc++ PROPERTIES
INTERFACE_INCLUDE_DIRECTORIES "${IMPORT_PREFIX}/include"
INTERFACE_LINK_LIBRARIES "wsock32;ws2_32;OpenSSL::SSL;OpenSSL::Crypto;protobuf::libprotobuf;gRPC::grpc;gRPC::gpr"
)
What did you see instead?
In gRPCTargets.cmake:
_set_target_properties(gRPC::grpc++ PROPERTIES
INTERFACE_INCLUDE_DIRECTORIES "${IMPORT_PREFIX}/include"
INTERFACE_LINK_LIBRARIES "wsock32;ws2_32;OpenSSL::SSL;OpenSSL::Crypto;libprotoc;libprotobuf;gRPC::grpc;gRPC::gpr"
)
In this case "libprotoc;libprotobuf" evolves into libs names(without absolute path). As result it is impossible to build project that use gRPC, because 'libprotoc.lib' not found.
The text was updated successfully, but these errors were encountered: