You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Mar 25, 2018. It is now read-only.
Repository navigation
This repository was archived by the owner on Mar 25, 2018. It is now read-only.
This was fixed for x64, but possibly never tested on x86. Now, x86 builds are failing (nodejs/build#669), the V8 update seems to be the root cause (still unconfirmed but the timing matches). The console output is similar:
The same commit that fixes this for x64 breaks for x86. I was able to confirm that this is because the change of position in v8.gyp, moving those 3 lines to the old position fixes this issue for x86.
Activity
targos commentedon Mar 8, 2017
Confirmed in a new run: https://ci.nodejs.org/job/node-compile-windows/7432/
/cc @nodejs/platform-windows
targos commentedon Mar 21, 2017
@joaocgreis FYI this issue is fixed in V8 master. I don't know what commit fixed it.
targos commentedon Mar 23, 2017
The error disappears with this commit: v8@3297130.
targos commentedon Mar 23, 2017
Confirmed: https://ci.nodejs.org/job/node-test-commit/8640/
joaocgreis commentedon Mar 30, 2017
This was fixed for x64, but possibly never tested on x86. Now, x86 builds are failing (nodejs/build#669), the V8 update seems to be the root cause (still unconfirmed but the timing matches). The console output is similar:
joaocgreis commentedon Mar 31, 2017
The same commit that fixes this for x64 breaks for x86. I was able to confirm that this is because the change of position in
v8.gyp, moving those 3 lines to the old position fixes this issue for x86.75 remaining items
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: set MSVS .obj folders in gyp for V8
deps,win: increase msvs_shard in gyp for V8
deps,win: increase msvs_shard in gyp for V8
jaimecbernardo commentedon Jun 30, 2017
A fix for this issue landed on V8 upstream.
Original PR: nodejs/node#13959
V8's Code Review: https://chromium-review.googlesource.com/c/556599/
V8's Commit: https://chromium.googlesource.com/v8/v8/+/3bef2af6ef198b0cd1342680bf5d7d782dc71a46