Skip to content

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

Merged
climbfuji merged 2 commits intoJCSDA:spack-stack-devfrom
climbfuji:bugfix/freetype_zlibapi_depbuildenv
Jan 10, 2025
Merged

Follow-up bug fix in freetype when using external libz; add conflict for node-js@21: with [email protected]#499
climbfuji merged 2 commits intoJCSDA:spack-stack-devfrom
climbfuji:bugfix/freetype_zlibapi_depbuildenv

Conversation

@climbfuji
Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

Thanks @srherbener!

@climbfuji climbfuji merged commit dd99fc1 into JCSDA:spack-stack-dev Jan 10, 2025
@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

No open projects

Development

Successfully merging this pull request may close these issues.

2 participants