Skip to content

release/1.6.0: bug fix for in py-gevent to build with Intel / oneAPI compilers#386

Merged
climbfuji merged 1 commit intoJCSDA:release/1.6.0from
climbfuji:bugfix/py_gevent_intel
Jan 2, 2024
Merged

release/1.6.0: bug fix for in py-gevent to build with Intel / oneAPI compilers#386
climbfuji merged 1 commit intoJCSDA:release/1.6.0from
climbfuji:bugfix/py_gevent_intel

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Dec 29, 2023

Description

See spack mainline PR for a description: spack#41896

py-gevent fails to build with some versions of Intel / oneAPI compilers (we've observed this on some but not all systems, with Intel versions ranging from intel@18.x.y to the latest oneAPI (classic) compilers.
I recently fixed a similar build error for curl (spack#41380), and conveniently the same method works for py-gevent. I tested this successfully on one of the Navy's HPCs.

@climbfuji
Copy link
Copy Markdown
Collaborator Author

The upstream spack develop PR was merged (spack#41896), this PR is ready to go.

Copy link
Copy Markdown
Collaborator

@ulmononian ulmononian left a comment

Choose a reason for hiding this comment

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

looks good!!!

@climbfuji climbfuji merged commit 46e4875 into JCSDA:release/1.6.0 Jan 2, 2024
@climbfuji climbfuji deleted the bugfix/py_gevent_intel branch January 2, 2024 20:11
climbfuji added a commit to JCSDA/spack-stack that referenced this pull request Jan 3, 2024
Update site configs and documentation for spack-stack-1.6.0 for:

Hercules
Nautilus
Narwhal (Intel only so far)
S4
Discover
Also: submodule pointer update for spack for a PR required to build on these platforms (spack PR JCSDA/spack#386 already merged).

Note the workaround for Narwhal that uses an external node.js that is supplied via the compiler config while the correct solution (for after release 1.6.0) is being worked on.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is not working INFRA JEDI Infrastructure

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants