Skip to content
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 Spack environment specification #838

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

stephenswat
Copy link
Member

This commit adds a YAML-based Spack environment specification which allows users to more easily install traccc's dependencies.

@stephenswat stephenswat added feature New feature or request build This relates to the build system labels Feb 4, 2025
Copy link
Member

@krasznaa krasznaa left a comment

Choose a reason for hiding this comment

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

It doesn't hurt, so sure...

@paulgessinger
Copy link
Member

Should we add this to the build cache populater as well at some point?

@stephenswat
Copy link
Member Author

Should we add this to the build cache populater as well at some point?

I think that would be a good idea!

Copy link
Contributor

@beomki-yeo beomki-yeo left a comment

Choose a reason for hiding this comment

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

Using -DTRACCC_USE_SYSTEM_LIBS=ON flag is mandatory?

Can we let spack install compilers, CMake, Boost and ROOT and let traccc CMake install other dependencies?

@stephenswat
Copy link
Member Author

Using -DTRACCC_USE_SYSTEM_LIBS=ON flag is mandatory?

Can we let spack install compilers, CMake, Boost and ROOT and let traccc CMake install other dependencies?

It's certainly not mandatory, but I don't see a good reason not to use it. If you have a package manager set up already there is not really any use in having CMake download the dependencies for you, right? 🤔

But yes, it is of course possible to install only the compilers, CMake, Boost, and ROOT with Spack and to use CMake for the rest if you prefer!

@beomki-yeo
Copy link
Contributor

I thought it would be bothersome to keep the versions of all dependencies updated. 🤔
I don't have a strong opinion on this so probably you can go with the setup you prefer.

@stephenswat stephenswat force-pushed the build/spack branch 2 times, most recently from b231412 to 42aed92 Compare February 21, 2025 13:47
@stephenswat
Copy link
Member Author

I changed this slightly, taking out the less stable dependencies for now (detray, vecmem, etc.).

This commit adds a YAML-based Spack environment specification which
allows users to more easily install traccc's dependencies.
@stephenswat stephenswat merged commit 3bad7fb into acts-project:main Feb 24, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build This relates to the build system feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants