Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add opencv3 check #114

Merged
merged 2 commits into from
Jan 8, 2018
Merged

Add opencv3 check #114

merged 2 commits into from
Jan 8, 2018

Conversation

allenh1
Copy link
Contributor

@allenh1 allenh1 commented Jan 7, 2018

Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

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

I'm not completely familiar with it but it looks fine. Maybe a link to the documentation statement about the gcc version support would be helpful. In case it changes in the future.

@allenh1
Copy link
Contributor Author

allenh1 commented Jan 8, 2018

@tfoote Struggling to find a good link, but I'll keep looking. This stackoverflow post confirms it, but I'd prefer a more formal source.

@allenh1 allenh1 merged commit b7c9f94 into master Jan 8, 2018
@allenh1 allenh1 deleted the add-opencv3-check branch January 8, 2018 04:02
zffgithub pushed a commit to zffgithub/superflore that referenced this pull request Apr 11, 2023
* Turn off cuda for opencv3 if GCC > 4. This also simplifies the logic for special cases.
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GCC6 and emerging opencv3.3 fails
3 participants