-
Notifications
You must be signed in to change notification settings - Fork 39
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
layers: move _POSIX_C_SOURCE where pthread_condattr_setclock is used #207
Conversation
Author jbeich not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
Author jbeich not on autobuild list. Waiting for curator authorization before starting CI build. |
I've already signed CLA in KhronosGroup/Vulkan-Loader#419 and KhronosGroup/Vulkan-ValidationLayers#5098. CLA assistant is still listed in https://github.com/settings/applications (Authorized OAuth Apps tab). |
Author jbeich not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
Author jbeich not on autobuild list. Waiting for curator authorization before starting CI build. |
CI Vulkan-ExtensionLayer build queued with queue ID 5245. |
CI Vulkan-ExtensionLayer build # 410 running. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the fix!
CI Vulkan-ExtensionLayer build # 410 passed. |
@jbeich Your fix is ready to go but you must sign the CLA before I can merge it. |
The CLA doesn't apply to the organization. It's configured per repo. Not sure why. |
https://cla-assistant.io/check/KhronosGroup/Vulkan-ExtensionLayer?pullRequest=207 redirects me to https://github.com/ (main page). Am I supposed to revoke @CLAassistant from authorized OAuth apps just to trigger CLA? |
This link looks fine. The check variant redirects. |
Thanks for the pointer. I've signed the CLA. |
Fixes #206