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
add integratorxx to buildsys #3041
Conversation
-DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER} | ||
-DCMAKE_INSTALL_LIBDIR=${CMAKE_INSTALL_LIBDIR} | ||
-DCMAKE_INSTALL_INCLUDEDIR=${CMAKE_INSTALL_INCLUDEDIR} | ||
-DCMAKE_CXX_STANDARD=${psi4_CXX_STANDARD} |
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.
@loriab FWIW - I don't respect this flag in IntegratorXX
- it's a C++17 library, you could conceivably compile with 20 / 2b, but definitely not with 14. Would you prefer that I have a check on my end if you're requesting < 17 and bail? Or should ENABLE_IntegratorXX
be contingent on 17+?
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.
all of Psi4 is 17, so in practice it's not an issue?
(only reason parts of psi could be 14 is if someone needed to use a ~3yo icpc with Libint2.)
Description
for now must enable with-D ENABLE_IntegratorXX=ON
. recc installing catch2 first to avoid downloading each time.CI will fail because I added a integratorxx header to a file for trial, but haven't enabled the addon in the buildsys everywhere.User API & Changelog headlines
-D ENABLE_IntegratorXX=ON
Checklist
Status