Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing artifacts for 1.83.0.23277 and 1.82.3.23277 #1669

Closed
daiyam opened this issue Oct 4, 2023 · 19 comments · Fixed by #1689
Closed

Missing artifacts for 1.83.0.23277 and 1.82.3.23277 #1669

daiyam opened this issue Oct 4, 2023 · 19 comments · Fixed by #1689

Comments

@daiyam
Copy link
Member

daiyam commented Oct 4, 2023

Hello,

Some artifacts are missing from the 1.83.0.23277 and 1.82.3.23277 releases:

  • macOS x86
  • windows (all)

Same error on both cases:

error D:\a\vscodium\vscodium\vscode\node_modules\@parcel\watcher: Command failed.
Exit code: 1
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
Command: node-gyp-build
Arguments: 
Directory: D:\a\vscodium\vscodium\vscode\node_modules\@parcel\watcher
Output:
node:events:495
      throw er; // Unhandled 'error' event
      ^

Error: spawn node-gyp.cmd ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:284:19)
    at onErrorNT (node:internal/child_process:477:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:290:12)
    at onErrorNT (node:internal/child_process:477:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn node-gyp.cmd',
  path: 'node-gyp.cmd',
  spawnargs: [ 'rebuild' ]
}

https://github.com/VSCodium/vscodium/actions/runs/6408642071/job/17398088692
https://github.com/VSCodium/vscodium/actions/runs/6409117121/job/17399626498

On local macOS x86, another dependency doesn't want to build.
On local windows, no issues.

@daiyam daiyam pinned this issue Oct 4, 2023
@daiyam daiyam changed the title Missing artifacts from 1.82.3.23277 and Insiders Missing artifacts for 1.83.0.23277 and 1.82.3.23277 Oct 4, 2023
@leefordjudes
Copy link

Kindly add binaries (1.83) for windows

@daiyam
Copy link
Member Author

daiyam commented Oct 7, 2023

Sorry, I don't have time, busy in RL.

@daiyam
Copy link
Member Author

daiyam commented Oct 9, 2023

I've tested few things but nothing work out. I've tested with the latest Windows image from GitHub, still the same.

@sid-the-sloth
Copy link

Please add back the AppImage build.

@daiyam
Copy link
Member Author

daiyam commented Oct 9, 2023

@sid-the-sloth see #1654...

@daiyam
Copy link
Member Author

daiyam commented Oct 9, 2023

Oh shit! The issue is due to node v18.18! Using v18.17 seems to fix the issue! #1689 (finally done a PR after all the shit I've done in the master...)

@daiyam
Copy link
Member Author

daiyam commented Oct 9, 2023

1.83.0.23277 seems to be complete!

@daiyam
Copy link
Member Author

daiyam commented Oct 9, 2023

I will leave 1.82.3.23277 as it is.

@daiyam daiyam closed this as completed Oct 9, 2023
@daiyam daiyam unpinned this issue Oct 9, 2023
@syndenbock
Copy link

Unfortunately VSCodium (win32-x64) no longer launches for me on this release.
I'm not sure if it's related to the new build settings and I don't have VS Code installed so I didn't open a separate issue.

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

@syndenbock It's not only you!

  • windows x64: broken
  • macos x64: ok

Edit: I've renamed the Windows binaries with --broken--.

Damn!

@daiyam daiyam reopened this Oct 10, 2023
@daiyam daiyam pinned this issue Oct 10, 2023
@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

Here the error Loading "@vscode/policy-watcher" failed

@vscode/sqlite3 and @parcel/watcher aren't there either. And maybe others. I will check that out tomorrow.

Edit:
The node_modules.asar is missing the built .node files of all the dependencies...

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

On local windows:

  • node v18.18: .node are built
  • node v18.17: .node are not built

No errors for either versions...

This is why the current issue is "fixed" with v18.17...
Weirdly macOS isn't affected by this problem. ???

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

I'm woking on a fix but for the Insiders version (#1690). When validated, I will make it for the regular one.

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

The Insiders versions are out and are working correctly.

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

I'm re-releasing the fixed version as 1.83.0.23283. It should out in about an hour.

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

There were 2 issues:

  • node v18.18 generate errors on Windows and macOS x64
  • on Windows, bad config of custom node-gyp for the spectre mitigation patch (the way to configure have changed)

@daiyam
Copy link
Member Author

daiyam commented Oct 10, 2023

Validated, Windows x64 and macOS x64 of 1.83.0.23283 are launching fine!

@peter-fb
Copy link

I just upgraded, Windows 10 x64, it seems to work fine. Thank you! :)

@syndenbock
Copy link

Oh shit! The issue is due to node v18.18! Using v18.17 seems to fix the issue! #1689 (finally done a PR after all the shit I've done in the master...)

A new LTS version 18.18.1 of node was released 5 hours ago including a fix for node images giving an error.
Could this have been the issue here so it works with 18.18.x again?

@daiyam daiyam closed this as completed Oct 15, 2023
@daiyam daiyam unpinned this issue Oct 15, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants