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

cleanup: rustc doesn't use an external archiver #45102

Merged
merged 1 commit into from
Oct 14, 2017

Conversation

petrochenkov
Copy link
Contributor

# is not the linker used to link said compiler.
# The default linker that will be hard-coded into the generated compiler for
# targets that don't specify linker explicitly in their target specifications.
# Note that this is not the linker used to link said compiler.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A drive-by comment improvement.

@bors
Copy link
Contributor

bors commented Oct 9, 2017

☔ The latest upstream changes (presumably #45041) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton
Copy link
Member

r=me

Thanks!

@carols10cents carols10cents added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Oct 9, 2017
@petrochenkov
Copy link
Contributor Author

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Oct 9, 2017

📌 Commit b434c84 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Oct 11, 2017

⌛ Testing commit b434c84 with merge 818c288b0ed69e03993c74ef8b45ebf077c8e216...

@bors
Copy link
Contributor

bors commented Oct 11, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Oct 11, 2017

@bors retry

@bors
Copy link
Contributor

bors commented Oct 14, 2017

⌛ Testing commit b434c84 with merge 83922fe...

bors added a commit that referenced this pull request Oct 14, 2017
cleanup: rustc doesn't use an external archiver

cc #45090
r? @alexcrichton
@bors
Copy link
Contributor

bors commented Oct 14, 2017

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

@bors bors merged commit b434c84 into rust-lang:master Oct 14, 2017
@petrochenkov petrochenkov deleted the noar branch June 5, 2019 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants