Skip to content

Update OpenSSL download location#42220

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:update
May 25, 2017
Merged

Update OpenSSL download location#42220
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:update

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

In rustbuild itself we download from our mirror but in the containers we don't
do this yet. The OpenSSL download url changes from time to time (it breaks when
they release a new version) so let's download from our mirror instead.

In rustbuild itself we download from our mirror but in the containers we don't
do this yet. The OpenSSL download url changes from time to time (it breaks when
they release a new version) so let's download from our mirror instead.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton

Copy link
Copy Markdown
Member Author

discovered on beta

@brson

brson commented May 25, 2017

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented May 25, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 357133b has been approved by brson

@alexcrichton

Copy link
Copy Markdown
Member Author

@bors: p=100

@bors

bors commented May 25, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 357133b with merge 5f39668...

bors added a commit that referenced this pull request May 25, 2017
Update OpenSSL download location

In rustbuild itself we download from our mirror but in the containers we don't
do this yet. The OpenSSL download url changes from time to time (it breaks when
they release a new version) so let's download from our mirror instead.
@bors

bors commented May 25, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 5f39668 to master...

@bors bors merged commit 357133b into rust-lang:master May 25, 2017
@alexcrichton alexcrichton deleted the update branch June 14, 2017 18:40
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

Successfully merging this pull request may close these issues.

5 participants