Skip to content

forget to look in sub folder of include/openblas#17729

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
hkinke:3.4
Jul 4, 2020
Merged

forget to look in sub folder of include/openblas#17729
opencv-pushbot merged 1 commit intoopencv:3.4from
hkinke:3.4

Conversation

@hkinke
Copy link
Copy Markdown

@hkinke hkinke commented Jul 2, 2020

When provide environment variable OpenBLAS_HOME, OpenCVFindOpenBLAS.cmake forgets to look in ${OpenBLAS_HOME}/include/openblas for include files.

@mshabunin
Copy link
Copy Markdown
Contributor

How do you install OpenBLAS? The Windows package from official OpenBLAS SourceForge does not have include/openblas subdirectory for example.

@hkinke
Copy link
Copy Markdown
Author

hkinke commented Jul 2, 2020

see #17679 . I already made a PR for same issue. I did it on the bad branch. For answering for your question, I build OpenBLAS on windows machine with cmake. I set an environment variable OpenBlas_HOME to the folder containing the installation of OpenBLAS. @alalek has accepted my PR. I choose to undo the first commit and to redo it on appropriate branch.

@opencv-pushbot opencv-pushbot merged commit 5a15369 into opencv:3.4 Jul 4, 2020
@alalek alalek mentioned this pull request Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants