Skip to content

chore: disable inherited PGO in Windows CI - #1763

Open
vmoroz wants to merge 1 commit into
nodejs:mainfrom
vmoroz:PR/fix-windows-addon-pgo
Open

vmoroz wants to merge 1 commit into
nodejs:mainfrom
vmoroz:PR/fix-windows-addon-pgo

Conversation

@vmoroz

@vmoroz vmoroz commented Nov 6, 2026

Copy link
Copy Markdown
Member

Windows CI

Disable Node.js 26's inherited PGO setting while compiling Windows addons. Its unquoted profile path otherwise breaks the addon_build test when the fixture directory contains spaces.

Apply the workaround at workflow scope so nested npm builds inherit it without changing Node's build settings.

Testing

Targeted native builds and runtime checks passed in both API modes. Full suite not run.

Disable Node.js 26's inherited PGO setting while compiling Windows
addons. Its unquoted profile path otherwise breaks the addon_build
test when the fixture directory contains spaces.

Apply the workaround at workflow scope so nested npm builds inherit
it without changing Node's build settings.

Signed-off-by: Vladimir Morozov <[email protected]>
Assisted-by: Copilot
Co-authored-by: Copilot <[email protected]>

This branch has not been deployed

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

Labels

None yet

Projects

Status: Need Triage

Development

Successfully merging this pull request may close these issues.

1 participant