fletch icon indicating copy to clipboard operation
fletch copied to clipboard

Dev/fix opencv gotoblas

Open Erotemic opened this issue 6 years ago • 9 comments

OpenCV 3.4 has an option OpenBLAS_LIB. I'm not sure if it exists in previous versions. ~~I need to check that in order to correctly set the version if statement~~ ( Turns out OpenBLAS support was added in 3.2, so I happened to guess right. ). The first function of this PR is to pass down that info when it is needed.

The second function is to fix an issue I get:

    [ 97%] Linking CXX executable ../../bin/opencv_traincascade
    //usr/lib/liblapack.so.3: undefined reference to `gotoblas`
    collect2: error: ld returned 1 exit status
    apps/traincascade/CMakeFiles/opencv_traincascade.dir/build.make:392: recipe for target 'bin/opencv_traincascade' failed

A workaround for this seems to be to set WITH_LAPACK=False. It would be better if I understood the issue more, but I don't.

Erotemic avatar Mar 01 '18 00:03 Erotemic

I was able to get OpenCV to depend on OpenBLAS by moving the OpenBLAS tarball block to appear before OpenCV.

Erotemic avatar Mar 01 '18 17:03 Erotemic

Jenkins test this please

dstoup avatar Mar 02 '18 17:03 dstoup

Mac build succeeded: https://open.cdash.org/buildSummary.php?buildid=5280095

kwcvrobot avatar Mar 02 '18 22:03 kwcvrobot

Linux build succeeded: https://open.cdash.org/buildSummary.php?buildid=5280882

kwcvrobot avatar Mar 03 '18 10:03 kwcvrobot

FYI, Windows didn't fail, I just stopped it while we examine review comments. We have a stack of Windows builds to get through.

dstoup avatar Mar 19 '18 11:03 dstoup

Jenkins test this please

dstoup avatar Mar 28 '18 11:03 dstoup

Mac build succeeded: https://open.cdash.org/buildSummary.php?buildid=5316211

kwcvrobot avatar Mar 28 '18 14:03 kwcvrobot

Linux build succeeded: https://open.cdash.org/buildSummary.php?buildid=5316371

kwcvrobot avatar Mar 28 '18 16:03 kwcvrobot