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

Add curl to all playbooks #404

Closed
smlambert opened this issue Jul 17, 2018 · 8 comments
Closed

Add curl to all playbooks #404

smlambert opened this issue Jul 17, 2018 · 8 comments

Comments

@smlambert
Copy link
Contributor

curl is present in https://github.com/AdoptOpenJDK/openjdk-infrastructure/tree/master/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Common/vars:
Ubuntu.yml
SLES.yml
Redhat.yml
FreeBSD.yml

but not in:
OracleLinux.yml
CentOS.yml

Should also ensure its present in the non-Linux playbooks.

@jdekonin
Copy link
Contributor

Is there a specific version that is required based on options needed?

fyi @llxia

@smlambert
Copy link
Contributor Author

curl (version 7.20.0 or later) to get the option -J/--remote-header-name

@vsebe
Copy link
Contributor

vsebe commented Oct 22, 2018

It is not required on Windows - using the cygwin package: curl 7.59.0-1

@sxa
Copy link
Member

sxa commented Oct 29, 2018

@smlambert For reference, what is that option needed for?

@smlambert
Copy link
Contributor Author

We wish to download binaries using the api, which is not a direct link to a tgz file, but redirects to one, the -J option is needed.

@sxa sxa modified the milestones: 2018 October B, 2018 November A Oct 30, 2018
@sxa
Copy link
Member

sxa commented Nov 6, 2018

Never noticed that wasn't default behaviour - good to know - I tend to use wget more than curl myself :-)

@sxa sxa added this to the 2018 November A milestone Nov 6, 2018
@smlambert
Copy link
Contributor Author

we were using wget too, but since we had several people request we change to use curl, and since wget is not available on z/os and we decided to switch to it (and trying to eliminate our use of both, to decrease the number of prereqs needed)

@jdekonin
Copy link
Contributor

Comment add to #554 for macos once that playbook is created. The only remaining one not already mentioned is for aix, which is installed as part of coreutils v7.60. I believe this is complete and can be closed.

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

No branches or pull requests

5 participants