-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[intel-mkl] Update description field #19461
[intel-mkl] Update description field #19461
Conversation
Co-authored-by: Phoebe <20694052+PhoebeHui@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 57bd7102d9fd880daa1b0958692294c4a125f6d8 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/i-/intel-mkl.json b/versions/i-/intel-mkl.json
index 36f844b..1c5ffb0 100644
--- a/versions/i-/intel-mkl.json
+++ b/versions/i-/intel-mkl.json
@@ -1,7 +1,7 @@
{
"versions": [
{
- "git-tree": "88fbc68eed77db424c50b2d4796a6478524ba0c7",
+ "git-tree": "f2d5b828df130d8b1106aa0c076eb9cb79e50ea9",
"version": "2020.0.0",
"port-version": 2
},
Thanks for your suggestion. @Neumann-A Since https://registrationcenter.intel.com/en/products/download/3178/ is also used in portfile.cmake, it would be better to keep the consistent with this in vcpkg.json. |
Thanks for your contribution! |
Describe the pull request
What does your PR fix?
Fixes [intel-mkl] build failure using GitHub Actions (MSBuild based project) #18896Add the information that users need to install MKL manually to
description
field.Note that please download and install MKL from https://registrationcenter.intel.com/en/products/download/3178/ manually before using it in vcpkg.
Note: No feature needs to test.