Skip to content

New urls for Kokkos and Kokkos-Kernels - #47330

Merged
alalazo merged 2 commits into
spack:developfrom
cedricchevalier19:new-urls
Nov 4, 2024
Merged

New urls for Kokkos and Kokkos-Kernels#47330
alalazo merged 2 commits into
spack:developfrom
cedricchevalier19:new-urls

Conversation

@cedricchevalier19

Copy link
Copy Markdown
Contributor

Since version 4.4, Kokkos and Kokkos-Kernels have released signed tarballs.
This PR makes Spack use them instead of standard Github-generated artifacts.

@nmm0 nmm0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, could probably get rid of explicit URLs

Comment thread var/spack/repos/builtin/packages/kokkos/package.py
@cedricchevalier19

Copy link
Copy Markdown
Contributor Author

@bernhardkaindl, can you check if it fits Spack policy and merge if so?

@bernhardkaindl bernhardkaindl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@bernhardkaindl, can you check if it fits Spack policy and merge if so?

I'm sorry, I can't because I don't know all of them. However, as @alalazo is active now, I mention him in this comment.

Feel free to ask him or a spack maintainer that likely knows better than me.

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.

6 participants