Skip to content

Update OPENSSL_CC_powerpc64-unknown-linux-gnu#3648

Merged
bors merged 1 commit intorust-lang:masterfrom
cuviper:powerpc64-gcc4
Feb 4, 2017
Merged

Update OPENSSL_CC_powerpc64-unknown-linux-gnu#3648
bors merged 1 commit intorust-lang:masterfrom
cuviper:powerpc64-gcc4

Conversation

@cuviper
Copy link
Member

@cuviper cuviper commented Feb 4, 2017

With the new rust-slave-linux-cross:2017-02-02, powerpc64 no longer
uses gcc-5. I updated the Dockerfile at the time to reflect this, but
missed OPENSSL_CC_powerpc64-unknown-linux-gnu in Makefile.in.

With the new `rust-slave-linux-cross:2017-02-02`, powerpc64 no longer
uses gcc-5.  I updated the `Dockerfile` at the time to reflect this, but
missed `OPENSSL_CC_powerpc64-unknown-linux-gnu` in `Makefile.in`.
@rust-highfive
Copy link

r? @brson

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

@cuviper
Copy link
Member Author

cuviper commented Feb 4, 2017

Followup to #3637. Sorry I didn't catch this before, but I'm surprised -- @bors didn't either?

This should probably also go to the rust-1.16.0 for the beta.

@cuviper
Copy link
Member Author

cuviper commented Feb 4, 2017

Travis caches target/openssl, so it looks like CI didn't even try to build openssl again. Is there a way to clear that, so even openssl will be rebuilt with the new toolchains?

@alexcrichton
Copy link
Member

Nah unfortunately there's not a manual stamp to clean out the old caches, but should likely add one! In any case this looks good to me, so

@bors: r+

@bors
Copy link
Contributor

bors commented Feb 4, 2017

📌 Commit d5ba122 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Feb 4, 2017

⌛ Testing commit d5ba122 with merge d710222...

bors added a commit that referenced this pull request Feb 4, 2017
Update OPENSSL_CC_powerpc64-unknown-linux-gnu

With the new `rust-slave-linux-cross:2017-02-02`, powerpc64 no longer
uses gcc-5.  I updated the `Dockerfile` at the time to reflect this, but
missed `OPENSSL_CC_powerpc64-unknown-linux-gnu` in `Makefile.in`.
@bors
Copy link
Contributor

bors commented Feb 4, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing d710222 to master...

@bors bors merged commit d5ba122 into rust-lang:master Feb 4, 2017
@ehuss ehuss added this to the 1.17.0 milestone Feb 6, 2022
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.

6 participants