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

1.67 regressed process creation with cwd, ENOENT error #149716

Closed
jeremie-stripe opened this issue May 17, 2022 · 4 comments
Closed

1.67 regressed process creation with cwd, ENOENT error #149716

jeremie-stripe opened this issue May 17, 2022 · 4 comments
Assignees
Labels
insiders-released Patch has been released in VS Code Insiders
Milestone

Comments

@jeremie-stripe
Copy link

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version:
    Version: 1.67.1 (Universal)
    Commit: da15b6f
    Date: 2022-05-06T12:37:16.526Z (1 wk ago)
    Electron: 17.4.1
    Chromium: 98.0.4758.141
    Node.js: 16.13.0
    V8: 9.8.177.13-electron.0
  • OS Version:
    OS: Darwin x64 21.4.0

Steps to Reproduce:

The issue is reminiscent of #118267 which until 1.67 had not reappeared. The problem is similar to what is described in that original bug report with the same end result that trying to start a process with a command that's a relative path and setting a cwd on that process causes the syscall to return ENOENT.

Looking at recent changes that went into libuv/electron, I saw mention of a similar issue and related fix: libuv/libuv#3597 (comment)

@VSCodeTriageBot
Copy link
Collaborator

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.67.1. Please try upgrading to the latest version and checking whether this issue remains.

Happy Coding!

@jeremie-stripe
Copy link
Author

Testing with the current Insiders version this issue doesn't seem to appear:

Version: 1.68.0-insider (Universal)
Commit: 5f3e9c1
Date: 2022-05-17T05:18:39.842Z
Electron: 17.4.3
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: 9.8.177.13-electron.0
OS: Darwin x64 21.4.0

@jeremie-stripe
Copy link
Author

cc @deepak1556 on the above, thanks!

@deepak1556
Copy link
Contributor

Sorry for the breakage with stable release, upstream fix electron/electron#33870 landed in Electron 17.4.2 and insiders is currently on 17.4.3. Fix will be available with the 1.68 stable release.

/closedWith 624bbc4

@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label May 17, 2022
@VSCodeTriageBot VSCodeTriageBot added this to the May 2022 milestone May 17, 2022
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 18, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jul 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Development

No branches or pull requests

4 participants