Skip to content

sgpp: update dependency versions#43178

Merged
alalazo merged 5 commits intospack:developfrom
G-071:sgpp-update-dependencies
Mar 18, 2024
Merged

sgpp: update dependency versions#43178
alalazo merged 5 commits intospack:developfrom
G-071:sgpp-update-dependencies

Conversation

@G-071
Copy link
Copy Markdown
Contributor

@G-071 G-071 commented Mar 14, 2024

SGpp recently had some changes to make it compatible with Python 3.12. This PR changes its Spack package accordingly (dropping a patch for the master branch and changing/relaxing some version constraints).

# TODO Once distutils is removed from SGpp, limit patch to @:3.4.0
patch("disable_disutils_deprecation_warning.patch", when="^[email protected]:3.11")
patch("disable_disutils_deprecation_warning.patch", when="@:3.4.0 ^[email protected]:3.11")
patch("for_aarch64.patch", when="target=aarch64:")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you add a brief comment above this patch directive explaining:

  1. What is the effect of the patch
  2. Why it is needed
  3. Where it was taken from

?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have updated the description of the disable_disutils_deprecation_warning.patch and linked the relevant issue within sgpp.

For the aarch64.patch: This one is only changed by the PR since I moved it up a few lines in the package.py recipe (as I found it odd to not have it together with the other patches within the package). It was added quite a while ago to the spack package in #18968. Of course, I still added a brief description for it.

@alalazo alalazo self-assigned this Mar 14, 2024
@G-071 G-071 requested a review from alalazo March 14, 2024 13:06
Co-authored-by: Massimiliano Culpo <[email protected]>
@G-071 G-071 requested a review from alalazo March 15, 2024 11:05
@alalazo alalazo merged commit d5c1e16 into spack:develop Mar 18, 2024
mathomp4 pushed a commit to mathomp4/spack that referenced this pull request Mar 27, 2024
teaguesterling pushed a commit to teaguesterling/spack that referenced this pull request Jun 15, 2024
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