Skip to content

BUG: path failure for find_package config files #4

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

Merged

Conversation

hjmjohnson
Copy link
Contributor

CMake Error at src/lapack-install/lib64/cmake/lapacke-3.6.1/lapacke-config.cmake:13 (include):
include could not find load file:

src/lapack-install/lib64/cmake/lapacke-3.6.1/lapacke-targets.cmake

Call Stack (most recent call first):
CMakeLists.txt:26 (find_package)

-- src/lapack-install/lib64/cmake/lapacke-3.6.1
-- Configuring incomplete, errors occurred!
See also "src/lapackTest-bld/CMakeFiles/CMakeOutput.log".

The file was installed in:
src/lapack-install/lib/cmake/lapacke-3.6.1/lapacke-targets.cmake
^^ - Missing lib suffix of 64

CMake Error at src/lapack-install/lib64/cmake/lapacke-3.6.1/lapacke-config.cmake:13 (include):
  include could not find load file:

    src/lapack-install/lib64/cmake/lapacke-3.6.1/lapacke-targets.cmake
Call Stack (most recent call first):
  CMakeLists.txt:26 (find_package)

-- src/lapack-install/lib64/cmake/lapacke-3.6.1
-- Configuring incomplete, errors occurred!
See also "src/lapackTest-bld/CMakeFiles/CMakeOutput.log".

The file was installed in:
    src/lapack-install/lib/cmake/lapacke-3.6.1/lapacke-targets.cmake
                          ^^ - Missing lib suffix of 64
@hjmjohnson
Copy link
Contributor Author

hjmjohnson commented Jul 9, 2016

@julielangou This problem only shows up on machines where 64bit libraries are put in lib64.

@hjmjohnson
Copy link
Contributor Author

@julielangou This bug fix has been fairly well tested on my Mac and SUSE linux boxes. Is there anything else you would like me to do before it is merged?

Thanks,
Hans

@langou langou merged commit b7486d8 into Reference-LAPACK:master Jul 11, 2016
@langou
Copy link
Contributor

langou commented Jul 11, 2016

Thanks Hans, just reviewed the pull request, tested it and accepted it. Thanks a lot. Julien.

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.

3 participants