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

use Cargo.lock file from rust-src if present #166

Merged
merged 2 commits into from
Sep 6, 2017

Conversation

RalfJung
Copy link
Collaborator

This ensures we are using the exact versions of the packages that were used to build the libstd of this Rust release.

Cc #162

@RalfJung
Copy link
Collaborator Author

RalfJung commented Sep 5, 2017

This PR fixes the problem that for some weeks now, compiling the latest libstd with xargo is broken.

@japaric
Copy link
Owner

japaric commented Sep 5, 2017

Thanks, @RalfJung.

@homunkulus r+

@homunkulus
Copy link
Contributor

📌 Commit 517a614 has been approved by japaric

@homunkulus
Copy link
Contributor

⌛ Testing commit 517a614 with merge 517a614...

@homunkulus
Copy link
Contributor

💔 Test failed - status-appveyor

@japaric
Copy link
Owner

japaric commented Sep 5, 2017

@homunkulus r+

@homunkulus
Copy link
Contributor

📌 Commit b96e9ff has been approved by japaric

@homunkulus
Copy link
Contributor

⌛ Testing commit b96e9ff with merge b96e9ff...

@homunkulus
Copy link
Contributor

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

@RalfJung
Copy link
Collaborator Author

RalfJung commented Sep 6, 2017

Yay. :)

What is your usual release cycle, i.e. when will this fix be in a release?

@RalfJung RalfJung deleted the lock branch September 6, 2017 07:54
@japaric
Copy link
Owner

japaric commented Sep 6, 2017

@RalfJung v0.3.9 has been released.

@RalfJung
Copy link
Collaborator Author

RalfJung commented Sep 6, 2017

Wow, awesome, thanks :)

@jamesmunns
Copy link

Thanks @RalfJung and @japaric !

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.

4 participants