Skip to content

Conversation

susilehtola
Copy link
Member

Description

Updates CMakeLists to pull libxc 7.0.0

User API & Changelog headlines

  • RN 1
  • RN 2

Dev notes & details

  • Feature1
  • Feature2

Questions

  • Question1

Checklist

Status

  • Ready for review
  • Ready for merge

@@ -20,7 +20,7 @@ else()
# Default: use a stable release tarball of libxc. To use the
# development version of libxc, instead, comment the URL line,
# and uncomment the GIT lines.
URL https://gitlab.com/libxc/libxc/-/archive/6.2.2/libxc-6.2.2.tar.gz
URL https://gitlab.com/libxc/libxc/-/archive/7.0.0/libxc-7.0.0.tar.gz
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so either v6 or v7 should work at build-time?

iirc, libxc has SameMajorVersion for its cmake tolerant version range. since find_package(libxc 6.2) it'll likely reject 7, so we'll need to put in the extra logic psi-side to allow 6 or 7 like we had earlier in the year for allow 5 or 6. also, the codedeps.yaml at top level will likely need edits. these are mostly notes to myself and/or explanation if CI fails.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah the API hasn't changed, only the ABI.

@loriab
Copy link
Member

loriab commented Oct 12, 2024

I made a change that should have no effect except poking the CI. I want to check that it now pulls v7 from c-f and then fails the cmake detection b/c it wants v6.

@loriab
Copy link
Member

loriab commented Oct 12, 2024

Ok, failed as anticipated, and now I've edited the branch to allow v6 or v7. If this build ok, gtg.

@loriab loriab added this to the Psi4 1.10 milestone Oct 12, 2024
@susilehtola susilehtola added this pull request to the merge queue Oct 15, 2024
Merged via the queue into psi4:master with commit b309a9a Oct 15, 2024
6 checks passed
@loriab loriab mentioned this pull request Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants