Skip to content

[krampfkn] Spack exercise #2

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Menkalian
Copy link

No description provided.

@Menkalian
Copy link
Author

Implemented additional task 1: Adding the main branch as version.

@Menkalian
Copy link
Author

Menkalian commented Nov 27, 2024

Implemented additional task 2: Adding build options to spack and CMake. To test the build, you would need to change the repo + branch of the "main" version. I included a patch-file for doing that. I also included the patched file as package2.py. Usually that would not be scope for a PR like this.

I chose a very simple/brute option in the code for adhering to the build options (would be even better to change the available interfaces, so we can detect at build time missing build options), but since that is not the main scope, I hope it is ok I did it in a simpler fashion..

@IshaanDesai
Copy link
Member

Nice work 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants