Skip to content

Adding patch to Opencv (3.3.0-3.4.1) for fixing conflict between CUDA and OpenCV dnn header file#8502

Merged
adamjstewart merged 4 commits intospack:developfrom
samfux84:opencv_341_cuda_dnn_patch
Jun 18, 2018
Merged

Adding patch to Opencv (3.3.0-3.4.1) for fixing conflict between CUDA and OpenCV dnn header file#8502
adamjstewart merged 4 commits intospack:developfrom
samfux84:opencv_341_cuda_dnn_patch

Conversation

@samfux84
Copy link
Copy Markdown
Contributor

@samfux84 samfux84 commented Jun 18, 2018

Fixes #8461

When installing OpenCV 3.4.1 or older versions (I could reproduce the same behavior also with version 3.3.0) with +cuda and +dnn, then the compilation failed, because one of the header files from the OpenCV dnn module has the same name as a cuda header file and the cuda header file is included instead of the OpenCV one.

Issue #8461 contains more information on the problem.

@adamjstewart
Copy link
Copy Markdown
Member

Has this patch been submitted upstream?

@samfux84
Copy link
Copy Markdown
Contributor Author

@adamjstewart: I have opened an issue in the OpenCV github repo (opencv/opencv#11745). One of the OpenCV developers already fixed this issue in the develop branch along with many other commits. I just created a patch to fix the problem for 3.4.1 and older versions (since I could reproduce it also with 3.3.0, iI have used when='@3.3.0:3.4.1'), and posted the patch in the issue opencv/opencv#11745 but did not report it elsewhere.

@adamjstewart adamjstewart merged commit 46bf21f into spack:develop Jun 18, 2018
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.

2 participants