Skip to content

lammps: simplify url_for_version logic#39554

Merged
bernhardkaindl merged 1 commit intospack:developfrom
rbberger:lammps_stable_versions
Aug 24, 2023
Merged

lammps: simplify url_for_version logic#39554
bernhardkaindl merged 1 commit intospack:developfrom
rbberger:lammps_stable_versions

Conversation

@rbberger
Copy link
Copy Markdown
Member

now uses the stable_versions set to check whether a given version should download a "stable" tarball.

now uses the stable_versions set to check whether a given version should
download a "stable" tarball.
Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

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

Confirmed all 66 version sha256 with the change.

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.

Looks very good!

@bernhardkaindl bernhardkaindl merged commit e67dca7 into spack:develop Aug 24, 2023
dyokelson pushed a commit to dyokelson/spack that referenced this pull request Aug 24, 2023
Use the stable_versions variable to check stable versions
mpokorny pushed a commit to mpokorny/spack that referenced this pull request Sep 18, 2023
Use the stable_versions variable to check stable versions
@rbberger rbberger deleted the lammps_stable_versions branch October 17, 2023 15:27
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.

4 participants