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

Bug/curl #628

Merged
merged 4 commits into from Sep 17, 2015
Merged

Bug/curl #628

merged 4 commits into from Sep 17, 2015

Commits on Sep 15, 2015

  1. run-tests: use curl instead of wget

    wget on RHEL6 is too old to implement --method=HEAD.  curl works
    just fine.
    mdomsch committed Sep 15, 2015
    Configuration menu
    Copy the full SHA
    0e3cd57 View commit details
    Browse the repository at this point in the history
  2. more curl cleanups

    mdomsch committed Sep 15, 2015
    Configuration menu
    Copy the full SHA
    fc0e375 View commit details
    Browse the repository at this point in the history
  3. Use test_curl_HEAD() throughout, add --location flag

    In some places we were using the test_curl_HEAD() function, in others
    we were manually calling it.  Instead, use the helper function
    everywhere.
    
    Add --location flag to curl command so that if S3 returns us a 302
    redirect (which it may do for its own purposes), we follow it properly.
    mdomsch committed Sep 15, 2015
    Configuration menu
    Copy the full SHA
    daf6da9 View commit details
    Browse the repository at this point in the history

Commits on Sep 17, 2015

  1. run-tests: detect if curl is on path before using it

    This adds a function similar to the shell 'which' command to detect
    if curl is available, rather than assuming curl is available on
    a posix system.  Cleaner.
    mdomsch committed Sep 17, 2015
    Configuration menu
    Copy the full SHA
    2072260 View commit details
    Browse the repository at this point in the history