Skip to content

report: list envvars using uv_os_environ()

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/envvars into master

Note: This PR is a bit premature. It requires the libuv 1.31.0 update, which will happen later this week. Only the second commit (report: list envvars using uv_os_environ()) is relevant. libuv 1.31.0 includes a new API, uv_os_environ(), for iterating over the environment variables.

This PR simplifies the diagnostic report's code for listing environment variables by using uv_os_environ() instead of platform specific code.

cc: @nodejs/libuv

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports