-
-
Notifications
You must be signed in to change notification settings - Fork 254
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
Enable threads for Windows #5034
base: develop
Are you sure you want to change the base?
Conversation
May need input from @qkoziol |
The reason this is failing is that we're turning Pthreads on/off via the presence of the pthreads.h header, which is checked elsewhere in CMake. If we pick C11 threads here and then also have H5_HAVE_PTHREAD_H set, the ifdefs in H5TS are likely to be confused. The Autotools are probably fine since it looks like configure.ac is smarter about checking for the Pthreads header, but ideally there should be an H5_USE_POSIX_THREADS symbol that is set independently of H5_HAVE_PTHREADS_H and used in the H5TS code (and the other thread options should be renamed to USE vs HAVE, IMHO). If you do that here, be sure to alert @qkoziol since he'll have to update his branches. |
I like this idea. I'm tied up with something else for a few days, but could tackle it later this week or early next, f someone else doesn't get to it first. |
v2.0.0 uses C11.