Skip to content

[clapack] Fix FindLAPACK.cmake script#22303

Closed
ahojnnes wants to merge 4 commits intomicrosoft:masterfrom
ahojnnes:user/joschonb/clapack-cmake-config-fix
Closed

[clapack] Fix FindLAPACK.cmake script#22303
ahojnnes wants to merge 4 commits intomicrosoft:masterfrom
ahojnnes:user/joschonb/clapack-cmake-config-fix

Conversation

@ahojnnes
Copy link
Contributor

@ahojnnes ahojnnes commented Jan 3, 2022

Without this patch, the LAPACK::LAPACK and clapack::clapack are never defined.

Copy link

@github-actions github-actions bot left a 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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for clapack but no changes to version or port version.
-- Version: 3.2.1#19
-- Old SHA: bedf61cf9cf765a2c823e1f2427a16af6d8a711e
-- New SHA: ab15c55af259508496e0a8b09132b33dd8e8cf6a
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

@ahojnnes ahojnnes changed the title Fix FindLAPACK.cmake for clapack [clapack] Fix FindLAPACK.cmake script Jan 3, 2022
Copy link

@github-actions github-actions bot left a 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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for clapack but no changes to version or port version.
-- Version: 3.2.1#19
-- Old SHA: bedf61cf9cf765a2c823e1f2427a16af6d8a711e
-- New SHA: ab15c55af259508496e0a8b09132b33dd8e8cf6a
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 1085a57da0725c19e19586025438e8c16f34c890 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/c-/clapack.json b/versions/c-/clapack.json
index e223985..f0700bc 100644
--- a/versions/c-/clapack.json
+++ b/versions/c-/clapack.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "ab15c55af259508496e0a8b09132b33dd8e8cf6a",
+      "git-tree": "bedf61cf9cf765a2c823e1f2427a16af6d8a711e",
       "version-string": "3.2.1",
       "port-version": 19
     },

@BillyONeal
Copy link
Member

Needs to be considered in concert with #21479

@JackBoosY JackBoosY added the category:port-bug The issue is with a library, which is something the port should already support label Jan 4, 2022
@JackBoosY
Copy link
Contributor

Draft this PR until we decide to pick one of the relevant PRs.

@JackBoosY JackBoosY marked this pull request as draft May 7, 2022 06:34
@JackBoosY
Copy link
Contributor

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.

@JackBoosY JackBoosY closed this Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants