Skip to content

Conversation

devsnek
Copy link
Member

@devsnek devsnek commented Sep 3, 2025

Includes a change to op2 macro that enables & v8::Isolate
or &mut v8::Isolate as op argument.

For the time being debug info has been disabled on Windows,
because of running into PDB errors.

@bartlomieju bartlomieju changed the title upgrade feat: V8 14.0 Sep 3, 2025
@bartlomieju bartlomieju marked this pull request as ready for review September 5, 2025 20:25
Comment on lines 136 to 138
RUSTFLAGS: >-
-D warnings -Cdebuginfo=0 ${{ inputs.os == 'windows-2022-xl' &&
'-Clink-arg=/DEBUG:NONE' || '' }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if its the increased disk space or this /DEBUG:None flg that fixed it. I guess we'll know when this is removed.

Also I noticed in your early attempt to disable debug info @bartlomieju that the option is not really passed all the way down to the linker. In this commit 310ff73, the final linker command still contains /DEBUG - not sure why.

@bartlomieju bartlomieju merged commit 41e01ed into main Sep 8, 2025
18 checks passed
@bartlomieju bartlomieju deleted the upgrade branch September 8, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants