Skip to content

Dyninst: add checksums for all supported versions#34051

Merged
bernhardkaindl merged 1 commit intospack:developfrom
hainest:dyninst_checksums
Nov 24, 2022
Merged

Dyninst: add checksums for all supported versions#34051
bernhardkaindl merged 1 commit intospack:developfrom
hainest:dyninst_checksums

Conversation

@hainest
Copy link
Copy Markdown
Contributor

@hainest hainest commented Nov 22, 2022

This will confict with #34050, so I'll fix it once that has been merged.

@mwkrentel

@hainest hainest force-pushed the dyninst_checksums branch 2 times, most recently from e5bf2b3 to 1ba69c0 Compare November 22, 2022 22:05
@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 23, 2022

#34050 has been merged!

@alalazo alalazo self-assigned this Nov 23, 2022
@hainest
Copy link
Copy Markdown
Contributor Author

hainest commented Nov 23, 2022

@alalazo Thanks! I've rebased here, so it's ready for review when you have time.

Copy link
Copy Markdown
Contributor

@bernhardkaindl bernhardkaindl left a comment

Choose a reason for hiding this comment

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

Since it's only adding checksums, it's safe to approve and merge.

@bernhardkaindl bernhardkaindl merged commit 1d7393c into spack:develop Nov 24, 2022
@hainest hainest deleted the dyninst_checksums branch November 24, 2022 02:47
amd-toolchain-support pushed a commit to amd-toolchain-support/spack that referenced this pull request Feb 16, 2023
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.

3 participants