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

gtest and downloading on-demand from Google Code #302

Closed
gtaylor opened this issue Apr 23, 2015 · 3 comments
Closed

gtest and downloading on-demand from Google Code #302

gtaylor opened this issue Apr 23, 2015 · 3 comments

Comments

@gtaylor
Copy link

gtaylor commented Apr 23, 2015

I'm not sure if Google Code is running into issues or what, but for some reason we are seeing really spotty reliability between Google Compute Engine instances and the gtest download URL (http://googletest.googlecode.com/files/gtest-1.5.0.tar.bz2) today. From my home connection, I am able to reach and download the file.

While this will probably be fixed by someone else eventually, it does bring up the fact that protobuf has moved to Github, yet its build process depends on something sitting back over at Google Code (which is apparently having issues today, and is on death row).

This could be a gtest issue, but I'm going to drop it here since it's causing me pain with protobuf.

@ctiller
Copy link
Contributor

ctiller commented Aug 25, 2015

FWIW I just sent out a pull request for grpc (grpc/grpc#3050) to use the google/googletest package on GitHub, now that there is one.

@xfxyjwf
Copy link
Contributor

xfxyjwf commented Aug 25, 2015

So they finally moved to github. Protobuf should be updated to use: https://github.com/google/googlemock as well.

@xfxyjwf
Copy link
Contributor

xfxyjwf commented Jan 20, 2016

We should move to use submodule as well. This is tracked by another issue:
#236

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants