Skip to content

linux add timer api #2253

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
Jun 21, 2021
Merged

linux add timer api #2253

merged 1 commit into from
Jun 21, 2021

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