Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow-up bug fix in freetype when using external libz; add conflict for node-js@21: with [email protected] #499

Merged

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jan 9, 2025

Description

This PR contains two separate changes/commits:

  1. A follow-up to a previous PR (Workaround for freetype build error with external libz and spack-built pkg-config #496) for freetype when using an external libz. We also need to add the location of the external zlib.pc to PKG_CONFIG_PATH to the dependents of freetype.
  2. Add an incomplete bug fix and therefore an additional conflict for node-js@21: with one particular version of gcc, namely [email protected]. As noted in FTBFS, version 22.11, gcc 11.2 nodejs/node#55596, [email protected] seems to have a bug that prevents building newer node-js versions. See Installation issue: node-js@22 with [email protected] spack/spack#48492 for more information.

This PR is required for JCSDA/spack-stack#1410 and it was tested as part of it on three HPC systems used by the Navy (Atlantis, Narwhal, Nautilus).

@climbfuji climbfuji force-pushed the bugfix/freetype_zlibapi_depbuildenv branch from ce828eb to 1bfb0a5 Compare January 9, 2025 22:17
…tin/packages/node-js/package.py var/spack/repos/builtin/packages/node-js/wasm-compiler-gcc11p2.patch)

Since this bug fix is not sufficient, add a conflict for node-js@21: with [email protected]
@climbfuji climbfuji force-pushed the bugfix/freetype_zlibapi_depbuildenv branch from 1bfb0a5 to 53c39ca Compare January 9, 2025 22:21
@climbfuji climbfuji self-assigned this Jan 9, 2025
@climbfuji climbfuji added the bug Something is not working label Jan 9, 2025
@climbfuji climbfuji marked this pull request as ready for review January 9, 2025 22:23
@climbfuji climbfuji requested a review from srherbener January 10, 2025 12:25
@climbfuji
Copy link
Collaborator Author

Thanks @srherbener!

@climbfuji climbfuji merged commit dd99fc1 into JCSDA:spack-stack-dev Jan 10, 2025
15 checks passed
@climbfuji climbfuji deleted the bugfix/freetype_zlibapi_depbuildenv branch January 10, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
Development

Successfully merging this pull request may close these issues.

2 participants