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

the f macro is unused in some targets #1122

Merged
merged 1 commit into from
Nov 12, 2018
Merged

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Nov 10, 2018

No description provided.

@rust-highfive
Copy link

r? @alexcrichton

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

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 10, 2018

📌 Commit 9b74d99 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Nov 10, 2018

⌛ Testing commit 9b74d99 with merge ac2a692...

bors added a commit that referenced this pull request Nov 10, 2018
the f macro is unused in some targets
@bors
Copy link
Contributor

bors commented Nov 10, 2018

💔 Test failed - status-travis

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Nov 10, 2018

@alexcrichton this fails because the beta of x86_64-apple-darwin is still not shipped with rustup (even though the beta of i686-apple-darwin now started shipping again). I manually merged a PR before that had the same issue.

@alexcrichton
Copy link
Member

@bors: retry

Now there's a beta!

@bors
Copy link
Contributor

bors commented Nov 12, 2018

⌛ Testing commit 9b74d99 with merge 0d2d7d7...

bors added a commit that referenced this pull request Nov 12, 2018
the f macro is unused in some targets
@bors
Copy link
Contributor

bors commented Nov 12, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Nov 12, 2018

⌛ Testing commit 9b74d99 with merge 5fb08ff...

bors added a commit that referenced this pull request Nov 12, 2018
the f macro is unused in some targets
@bors
Copy link
Contributor

bors commented Nov 12, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Nov 12, 2018

⌛ Testing commit 9b74d99 with merge 5cc0066...

bors added a commit that referenced this pull request Nov 12, 2018
the f macro is unused in some targets
@bors
Copy link
Contributor

bors commented Nov 12, 2018

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

@bors bors merged commit 9b74d99 into rust-lang:master Nov 12, 2018
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