-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Copy download-or-bust helper from gce to aws #5014
Conversation
Spotted the |
I'd fix the helper function in this PR. |
OK - thanks: Fixed the message in the helper function as well. |
@@ -0,0 +1,29 @@ | |||
#!/bin/bash | |||
|
|||
# Copyright 2014 Google Inc. All rights reserved. |
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.
nit: 2015
Couple of nits; otherwise LGTM. |
e1ea065
to
758eed6
Compare
Don't merge yet ... I somehow touched the previous commit during my squash |
During a push, we were wget-ing to <filename>.1, .2, etc, which wasn't working
758eed6
to
05d4aea
Compare
OK, fixed up the git problems. Sorry about that. I made the proposed changes: fixed the copyright and removed the commented out detect-project. |
LGTM. |
Waiting for travis to pass before merging. |
Poking travis again.... |
Copy download-or-bust helper from gce to aws
During a push, we were wget-ing to .1, .2, etc, which wasn't working