GitHub user simonsouth opened a pull request:

    https://github.com/apache/thrift/pull/754

    THRIFT-3498 C++ library assumes optional function 
pthread_attr_setschedpolicy is available

    This patch allows Thrift to build on Haiku and other systems that do not 
implement POSIX's thread-execution scheduling option. It adds a check around 
the call to `pthread_attr_setschedpolicy` in the C++ library.
    
    [According to 
POSIX](http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/unistd.h.html#tag_13_77_03_02)
 `_POSIX_THREAD_PRIORITY_SCHEDULING` should be defined in `unistd.h` with a 
value greater than zero if the `pthread_attr_setschedpolicy` function is known 
to be available at runtime.
    
    `PosixThreadFactory.cpp`: Test for availability of optional 
`pthread_attr_setschedpolicy` function before including in source code

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/simonsouth/thrift 
thrift-3498-c++-library-assumes-optional-function

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/754.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #754
    
----
commit 69ae417ce33526457e41b8458f9c5d25e11f3992
Author: Simon South <[email protected]>
Date:   2015-12-21T19:01:36Z

    THRIFT-3498 C++ library assumes optional function 
pthread_attr_setschedpolicy is available
    
    PosixThreadFactory.cpp: Test for availability of optional
    pthread_attr_setschedpolicy function before including in source code

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to