Skip to content

Set CMAKE_MODULE_PATH appropriately to always use preferred linalg-modules#38

Closed
ajaypanyala wants to merge 11 commits into
wavefunction91:masterfrom
ajaypanyala:master
Closed

Set CMAKE_MODULE_PATH appropriately to always use preferred linalg-modules#38
ajaypanyala wants to merge 11 commits into
wavefunction91:masterfrom
ajaypanyala:master

Conversation

@ajaypanyala
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread CMakeLists.txt Outdated
Comment thread src/CMakeLists.txt
@wavefunction91
Copy link
Copy Markdown
Owner

@ajaypanyala Is there anything else you want to add to this? I'm going to hold off on merging until wavefunction91/linalg-cmake-modules#1 is merged

@ajaypanyala
Copy link
Copy Markdown
Contributor Author

I am done with my edits. Thanks!

Comment thread cmake/gauxc-dep-versions.cmake Outdated
@@ -1,5 +1,5 @@
set( GAUXC_LINALG_MODULES_REPOSITORY https://github.com/wavefunction91/linalg-cmake-modules.git )
set( GAUXC_LINALG_MODULES_REVISION 89c82b78505f60657ef33b5bf38e5da059dff7f2 )
set( GAUXC_LINALG_MODULES_REPOSITORY https://github.com/ajaypanyala/linalg-cmake-modules )
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Note before merging: This needs to change once wavefunction91/linalg-cmake-modules#1 is merged

Comment thread cmake/gauxc-dep-versions.cmake Outdated
set( GAUXC_LINALG_MODULES_REPOSITORY https://github.com/wavefunction91/linalg-cmake-modules.git )
set( GAUXC_LINALG_MODULES_REVISION 89c82b78505f60657ef33b5bf38e5da059dff7f2 )
set( GAUXC_LINALG_MODULES_REPOSITORY https://github.com/ajaypanyala/linalg-cmake-modules )
set( GAUXC_LINALG_MODULES_REVISION main )
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TODO: Pin commit before merging.

@ajaypanyala
Copy link
Copy Markdown
Contributor Author

am assuming this will be outdated soon. Will close it here and create a new PR later on devel repo if the changes in this PR are still required.

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.

2 participants