Skip to content

Conversation

@devnexen
Copy link
Contributor

No description provided.

@rust-highfive
Copy link

r? @m-ou-se

(rust-highfive has picked a reviewer for you, use r? to override)

@JohnTitor
Copy link
Member

r? @JohnTitor @bors r+

@bors
Copy link
Contributor

bors commented Jun 21, 2021

📌 Commit e2c2b34 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Jun 21, 2021

⌛ Testing commit e2c2b34 with merge a27c762...

bors added a commit that referenced this pull request Jun 21, 2021
@bors
Copy link
Contributor

bors commented Jun 21, 2021

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member

JohnTitor commented Jun 21, 2021

Hm, the lint now covers one more pattern (rust-lang/rust#86426), I'll handle it later.

@JohnTitor
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Jun 21, 2021

⌛ Testing commit e2c2b34 with merge 2c94600...

@bors
Copy link
Contributor

bors commented Jun 21, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 2c94600 to master...

@bors bors merged commit 2c94600 into rust-lang:master Jun 21, 2021
@12101111
Copy link
Contributor

12101111 commented Jul 5, 2021

Is #[link(name = "rt")] needed? It's already linked here:

libc/src/unix/mod.rs

Lines 310 to 311 in 90aa0e3

#[link(name = "rt", kind = "static-nobundle",
cfg(target_feature = "crt-static"))]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants