Skip to content

CI: use Xargo 0.2.x #120

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

Merged
merged 1 commit into from
Nov 24, 2016
Merged

CI: use Xargo 0.2.x #120

merged 1 commit into from
Nov 24, 2016

Conversation

japaric
Copy link
Member

@japaric japaric commented Nov 23, 2016

this version uses the rust-src component that rustup installs instead
of fetching the source tarball from static.r-l.o, which sometimes went
wrong due to the fallible logic that Xargo 0.1.x used

r? @alexcrichton

The issue is that we must now install rustup as well

this version uses the rust-src component that rustup installs instead
of fetching the source tarball from static.r-l.o, which sometimes went
wrong due to the fallible logic that Xargo 0.1.x used
@japaric
Copy link
Member Author

japaric commented Nov 23, 2016

which sometimes went wrong due to the fallible logic that Xargo 0.1.x used

This caused the error seen in #119

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 23, 2016

📌 Commit 5c0e0f2 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Nov 23, 2016

⌛ Testing commit 5c0e0f2 with merge 4d7f401...

bors added a commit that referenced this pull request Nov 23, 2016
CI: use Xargo 0.2.x

this version uses the rust-src component that rustup installs instead
of fetching the source tarball from static.r-l.o, which sometimes went
wrong due to the fallible logic that Xargo 0.1.x used

r? @alexcrichton

The issue is that we must now install rustup as well
@alexcrichton alexcrichton merged commit 8bb7795 into rust-lang:master Nov 24, 2016
@japaric japaric deleted the xargo-up branch November 29, 2016 16:19
tgross35 pushed a commit to tgross35/compiler-builtins that referenced this pull request Feb 23, 2025
120: test edge cases r=japaric a=japaric

the test generator is not good at generating test cases that contain values like inf, zero and nan.
This commit adds test cases that contain those values.

Co-authored-by: Jorge Aparicio <[email protected]>
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.

3 participants