Skip to content

android: add POSIX sysconf constants #990

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
May 22, 2018

Conversation

@rust-highfive
Copy link

r? @alexcrichton

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

@lucab
Copy link
Contributor Author

lucab commented May 22, 2018

/cc @asomers

I'm not sure why these were not included in #628

@asomers
Copy link
Contributor

asomers commented May 22, 2018

@lucab they weren't included simply because I didn't know which Android sdk to use as a reference.

@lucab
Copy link
Contributor Author

lucab commented May 22, 2018

I think it is safe to add them now. The oldest supported android API at the moment is from 2015, and this commit predates it. Worst case is that older bionic will return EINVAL on these.

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented May 22, 2018

📌 Commit 725608c has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented May 22, 2018

⌛ Testing commit 725608c with merge eeb3318...

@bors
Copy link
Contributor

bors commented May 22, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented May 22, 2018

⌛ Testing commit 725608c with merge 7d64d5a...

@bors
Copy link
Contributor

bors commented May 22, 2018

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

@bors bors merged commit 725608c into rust-lang:master May 22, 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.

5 participants