Skip to content

deps: add ngtcp2_fmt.c to build configuration (ngtcp2.gyp)#63821

Open
ShenHongFei wants to merge 1 commit into
nodejs:mainfrom
ShenHongFei:fix-ngtcp2-gyp
Open

deps: add ngtcp2_fmt.c to build configuration (ngtcp2.gyp)#63821
ShenHongFei wants to merge 1 commit into
nodejs:mainfrom
ShenHongFei:fix-ngtcp2-gyp

Conversation

@ShenHongFei

Copy link
Copy Markdown
Contributor

The following compilation configuration will result in an error.

python configure.py --v8-enable-temporal-support --experimental-quic --experimental-dtls --enable-thin-lto --dest-cpu=x64 --clang-cl=20.1.8
lld-link : warning : ignoring unknown argument '-fno-lto' [D:\1\nodejs\node_mksnapshot.vcxproj]
lld-link : warning : ignoring unknown argument '-flto=thin' [D:\1\nodejs\node_mksnapshot.vcxproj]
lld-link : error : undefined symbol: ngtcp2_fmt_write_str [D:\1\nodejs\node_mksnapshot.vcxproj]
  >>> referenced by ngtcp2\lib\ngtcp2_conn.c
  >>>               ngtcp2.lib(ngtcp2_conn.obj)
  >>> referenced by ngtcp2\lib\ngtcp2_pv.c
  >>>               ngtcp2.lib(ngtcp2_pv.obj)
  >>> referenced by ngtcp2\lib\ngtcp2_log.c
  >>>               ngtcp2.lib(ngtcp2_log.obj)
  >>> referenced 3 more times
  
lld-link : error : undefined symbol: ngtcp2_fmt_write_uint64w [D:\1\nodejs\node_mksnapshot.vcxproj]
  >>> referenced by ngtcp2\lib\ngtcp2_conn.c
  >>>               ngtcp2.lib(ngtcp2_conn.obj)
  >>> referenced by ngtcp2\lib\ngtcp2_pv.c
  >>>               ngtcp2.lib(ngtcp2_pv.obj)
  >>> referenced by ngtcp2\lib\ngtcp2_log.c
  >>>               ngtcp2.lib(ngtcp2_log.obj)
  >>> referenced 3 more times
  
...

After investigation, it was found that the sources section of the ngtcp2.gyp configuration file was missing the ngtcp2_fmt.c file, which caused the linking failure. After adding it, local compilation could be done normally.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/quic
  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3. labels Jun 9, 2026
@ShenHongFei

Copy link
Copy Markdown
Contributor Author

#63777

This PR has added this file.

image

@pimterry pimterry added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 9, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 9, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants