-
Notifications
You must be signed in to change notification settings - Fork 387
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
Fix build on OpenBSD #4231
base: master
Are you sure you want to change the base?
Fix build on OpenBSD #4231
Conversation
Related #4103 |
@catap I am not against having your suggestions on top of it. This has more to do with agreement within the team and reverting the appropriate change. |
My focus was not on restoring the previous code that supported multiple non-portable pthread functions. The purpose of my patch was to bring back ifdef guards against a non-portable function just in case it wouldn't exist on some systems. I turned out to be on the right track. |
@dridi thus, here a PR which is needed to be able to build varnish on OpenBSD again. |
@bsdphk can we leave the final verdict with you? |
I had just rebased this PR into the last master. Also, this fixes required for build Varnish on OpenBSD. |
Here a two trivial commit which unblock current master on OpenBSD.