Skip to content
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

Add additional test to ensure correct detection for threads #1151

Conversation

kdt3rd
Copy link
Contributor

@kdt3rd kdt3rd commented Sep 19, 2021

It would appear cmake doesn't preserve the prefer-pthread argument in
the generated find dependency call, also, ensure cmake is happy with the boolean
check.

Signed-off-by: Kimball Thurston kdt3rd@gmail.com

It would appear cmake doesn't preserve the prefer-pthread argument in
the find dependency call, also, ensure cmake is happy with the boolean
check.

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
@kdt3rd
Copy link
Contributor Author

kdt3rd commented Sep 19, 2021

This will address #1121

@cary-ilm
Copy link
Member

In the PR title, did you mean "additional test" instead of "additional text"?

@cary-ilm cary-ilm changed the title Add additional text to ensure correct detection for threads Add additional test to ensure correct detection for threads Sep 22, 2021
Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cary-ilm cary-ilm merged commit 91b8969 into AcademySoftwareFoundation:master Sep 22, 2021
cary-ilm pushed a commit to cary-ilm/openexr that referenced this pull request Sep 23, 2021
…oftwareFoundation#1151)

It would appear cmake doesn't preserve the prefer-pthread argument in
the find dependency call, also, ensure cmake is happy with the boolean
check.

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
cary-ilm pushed a commit that referenced this pull request Sep 29, 2021
It would appear cmake doesn't preserve the prefer-pthread argument in
the find dependency call, also, ensure cmake is happy with the boolean
check.

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
@kdt3rd kdt3rd deleted the add_addl_thread_check_after_install branch February 12, 2022 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants