Skip to content

Add clock_gettime related functions to macOS #777

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 2 commits into from
Sep 21, 2017

Conversation

ekmartin
Copy link
Contributor

Fixes #776. The constants were retrieved on macOS 10.12.6. Tried adding clock_nanosleep too, but it doesn't seem like macOS' libc includes that yet.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Sep 21, 2017

📌 Commit ffba97f has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 21, 2017

⌛ Testing commit ffba97f with merge 04412d0...

bors added a commit that referenced this pull request Sep 21, 2017
Add clock_gettime related functions to macOS

Fixes #776. The constants were retrieved on macOS 10.12.6. Tried adding `clock_nanosleep` too, but it doesn't seem like macOS' libc includes that yet.
@bors
Copy link
Contributor

bors commented Sep 21, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Sep 21, 2017

📌 Commit 1600470 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 21, 2017

⌛ Testing commit 1600470 with merge ecbd1fa...

bors added a commit that referenced this pull request Sep 21, 2017
Add clock_gettime related functions to macOS

Fixes #776. The constants were retrieved on macOS 10.12.6. Tried adding `clock_nanosleep` too, but it doesn't seem like macOS' libc includes that yet.
@ekmartin
Copy link
Contributor Author

ekmartin commented Sep 21, 2017

Hm, clock_settime isn't available on iOS. @alexcrichton: is there any way to include it only on macOS? The other two functions are available on iOS as well.

(EDIT: Sorry, amended away the commit that added the cfg_attr clause here - since the Travis build failed)

Doesn't work on iOS.
@alexcrichton
Copy link
Member

@bors: r+

Ah ok looks like travis is green so let's try again...

@bors
Copy link
Contributor

bors commented Sep 21, 2017

📌 Commit 585e752 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 21, 2017

⌛ Testing commit 585e752 with merge 4729bc1...

bors added a commit that referenced this pull request Sep 21, 2017
Add clock_gettime related functions to macOS

Fixes #776. The constants were retrieved on macOS 10.12.6. Tried adding `clock_nanosleep` too, but it doesn't seem like macOS' libc includes that yet.
@bors
Copy link
Contributor

bors commented Sep 21, 2017

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

@bors bors merged commit 585e752 into rust-lang:master Sep 21, 2017
@ekmartin ekmartin deleted the macos_gettime branch September 21, 2017 21:30
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