Skip to content

scotch: patch instead of conflict for oneAPI/FPE issue#939

Merged
alalazo merged 2 commits intospack:developfrom
AlexanderRichert-NOAA:scotch_oneapi
Aug 8, 2025
Merged

scotch: patch instead of conflict for oneAPI/FPE issue#939
alalazo merged 2 commits intospack:developfrom
AlexanderRichert-NOAA:scotch_oneapi

Conversation

@AlexanderRichert-NOAA
Copy link
Copy Markdown
Contributor

This PR eliminates the oneAPI conflict for Scotch, instead patching the file that leads to the FPE getting thrown when those flags are enabled.

Copy link
Copy Markdown
Contributor

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Can you post a link to the issue/discussion on GitHub for this solution? I find it VERY hard to "read" the patch. Thanks!

@AlexanderRichert-NOAA
Copy link
Copy Markdown
Contributor Author

Let me see what I can figure out. Unfortunately the code and discussion are pretty much all in the form of emails and a code tarball...

@climbfuji
Copy link
Copy Markdown
Contributor

Let me see what I can figure out. Unfortunately the code and discussion are pretty much all in the form of emails and a code tarball...

Never mind then. Would have been nice but not a showstopper.

@alalazo alalazo merged commit c750f99 into spack:develop Aug 8, 2025
17 checks passed
AlexanderRichert-NOAA added a commit to AlexanderRichert-NOAA/spack-packages that referenced this pull request Aug 8, 2025
AlexanderRichert-NOAA added a commit to JCSDA/spack-packages that referenced this pull request Aug 11, 2025
scotch: patch instead of conflict for oneAPI/FPE issue (spack#939)
hfbauman pushed a commit to hfbauman/spack-packages that referenced this pull request Aug 12, 2025
afzpatel pushed a commit to afzpatel/spack-packages that referenced this pull request Aug 12, 2025
mladenivkovic pushed a commit to mladenivkovic/spack-packages that referenced this pull request Aug 31, 2025
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