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
Add distribtest for using grpc as a pre-installed cmake package #12469
Conversation
|
007f99a
to
5d2b1b3
Compare
|
|
@@ -27,4 +27,6 @@ RUN apt-get update && apt-get install -y \ | |||
pkg-config \ | |||
unzip && apt-get clean | |||
|
|||
RUN apt-get update && apt-get install -y cmake golang && apt-get clean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious if golang
from jessie
is new enough or shall we fetch from golang.org tarballs and install that instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the golang
here is only to be able to build boringssl using cmake (they are using go to generate some code or config). It's not golang used by gRPC go (and I assume that's what you meant by 1.6 support going away soon).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (please ensure golang package version is at least 1.7, as 1.6 support has/will go away soon—preferably 1.8/9)
5d2b1b3
to
ae6fd66
Compare
|
|
Based on #12411.
Add add a distribtest that installs gRPC using cmake (with gRPC_XXX_PROVIDER=package) and then tests the helloworld example can be built using cmake.
Currently
make install
is a better way to install gRPC on Linux, but allowing using cmake install might be useful for other platforms (Windows) and adding the distribtest also revealed some flaws in our CMakeLists.txt.Followup work:
-- setup cmake distribtests where grpc is used as a submodule.
-- decide what cmake use cases will be officially supported (grpc=submodule, grpc=package, ...)