Skip to content

Add pthread_mutex_timedlock #411

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
Oct 3, 2016
Merged

Add pthread_mutex_timedlock #411

merged 1 commit into from
Oct 3, 2016

Conversation

apasel422
Copy link
Contributor

No description provided.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Sep 29, 2016

📌 Commit e42f9f0 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 29, 2016

⌛ Testing commit e42f9f0 with merge c7e08fc...

bors added a commit that referenced this pull request Sep 29, 2016
@alexcrichton
Copy link
Member

Looks like this may not exist on OSX?

@apasel422
Copy link
Contributor Author

Yeah, I have to investigate further.

@apasel422
Copy link
Contributor Author

What's the proper organization for including this in Linux and BSD but not OSX or iOS?

@alexcrichton
Copy link
Member

It's fine to duplicate the definition in the non-apple BSD modules

@apasel422
Copy link
Contributor Author

r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 3, 2016

📌 Commit 45179c9 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Oct 3, 2016

⌛ Testing commit 45179c9 with merge ed16fe6...

bors added a commit that referenced this pull request Oct 3, 2016
@bors
Copy link
Contributor

bors commented Oct 3, 2016

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

@bors bors merged commit 45179c9 into rust-lang:master Oct 3, 2016
@apasel422 apasel422 deleted the pthread_mutex_timedlock branch October 3, 2016 11:07
danielverkamp pushed a commit to danielverkamp/libc that referenced this pull request Apr 28, 2020
…target_feature (rust-lang#432)

* fix build after stabilization of cfg_target_feature and target_feature

* fix doc tests

* fix spurious unused_attributes warning

* fix more unused attribute warnings

* More unnecessary target features

* Remove no longer needed trait imports

* Remove fixed upstream workarounds

* Fix parsing the #[assert_instr] macro

Following upstream proc_macro changes

* Fix form and parsing of #[simd_test]

* Don't use Cargo features for testing modes

Instead use RUSTFLAGS with `--cfg`. This'll help us be compatible with the
latest Cargo where a tweak to workspaces and features made the previous
invocations we had invalid.

* Don't thread RUSTFLAGS through docker

* Re-gate on x86 verification

Closes rust-lang#411
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.

4 participants