-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
lapack-reference:x64-windows build failure #12949
Comments
This issue should be fixed by PR #12569, could you please ensure your vcpkg is latest? |
I just had the same issue with the most recent version of vcpkg. The problem seems to be that OpenBLAS does not install the pkg-config file when pkg-config is not found during configure time. See https://github.com/xianyi/OpenBLAS/blob/efdd237a91646f0ce58815ef6507c04e393813a6/CMakeLists.txt#L392-L396. I have opened a PR #12956 to address that issue. |
Vcpkg is the latest. The problem is caused by |
cmake version? |
Both 3.17.2 and 3.18.1 have the same error. |
in port file or triplet str
log file attach |
Console output
Log files |
/cc @Neumann-A
although no, there is a release check going on |
|
see #12956. The problem is probably openblas and not lapack. lapack just creates a symlink to/copy of openblas.pc file named blas.pc if and only if openblas.pc exists. |
Yes, that should be the root cause, |
I have the same problem Please any help? |
merge patch #12956 |
Could you please clarify what do you mean by "merge patch". I am using Windows 10 and running these command on cmd. Also this is the detail of the problem CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:74 (message): Perhaps you should add the directory containing `blas.pc' to the PKG_CONFIG_PATH environment variable Package 'blas', required by 'lapack', not found Error: Building package lapack-reference:x64-windows failed with: BUILD_FAILED |
примени патч, указанный в ссылке, что не понятного |
Host Environment
Windows 10 20H1
;MSVC 2019 16.7.1
.To Reproduce
Steps to reproduce the behavior:
Failure logs
Console output:
Additional context
It's not the same as #12888. The error happened after pull request Jimmy-Hu#285 had been merged.
The text was updated successfully, but these errors were encountered: