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

acts dependencies: new versions as of 2025/01/08 #48465

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

stephenswat
Copy link
Contributor

New year, new versions of the ACTS dependencies. This time, we have a new version of detray, a new version of algebra plugins, as well as a new version of ACTS itself.

Tagging @paulgessinger.

New year, new versions of the ACTS dependencies. This time, we have a
new version of detray, a new version of algebra plugins, as well as a
new version of ACTS itself.
@wdconinc wdconinc self-assigned this Jan 8, 2025
@@ -77,6 +78,7 @@ class Detray(CMakePackage):
depends_on("acts-algebra-plugins +vc", when="+vc")
depends_on("acts-algebra-plugins +eigen", when="+eigen")
depends_on("acts-algebra-plugins +smatrix", when="+smatrix")
depends_on("[email protected]:", when="@0.87:")
Copy link
Contributor

Choose a reason for hiding this comment

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

More an upstream comment, but where is this manifested in acts-project/detray@v0.86.0...v0.87.0?

Copy link
Contributor

@wdconinc wdconinc left a comment

Choose a reason for hiding this comment

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

Verified checksums and hashes. Checked CMakeLists.txt. One comment that probably more points to the absence of an explicit version requirement upstream, rather than something that is incorrect in this PR. I will mark as automerge, so if you want just mark that comment as resolved and it will merge.

@wdconinc wdconinc enabled auto-merge (squash) January 8, 2025 17:23
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