-
Notifications
You must be signed in to change notification settings - Fork 29k
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
Relaunching insiders with custom CLI Args fails due to missing 1.39.0-insider-main.sock #81652
Comments
@DonJayamanne does not reproduce with stable? |
No it doesn't |
@DonJayamanne I cannot reproduce, but I wonder how you start code from the command line. Are you literally putting in the path to the application bundle? Why not just @joaomoreno I wonder what could be going on here:
// it happens on Linux and OS X that the pipe is left behind
// let's delete it, since we can't connect to it and then
// retry the whole thing But there is no retry? Did we maybe loose this code during some refactoring? |
I launch it manually, as this isn't my system wide install of insiders. Basically these are part of our smoke tests, downloading VSCode insiders and launching it with custom args. |
@DonJayamanne don't you have to put a |
Sorry, i removed that bit just to make it more readable, here's the full command line:
|
@DonJayamanne ok thanks, still does not reproduce for me. If it is Electron related, we just reverted back to Electron 4. Can you reproduce in todays insider? |
Still the same problem. |
No, still looks the same. We try to delete it. If we succeed to delete it, we were correct in assuming that the pipe was left behind, so we just retry the whole IPC setup. If we can't delete it (and we can't connect to it)... something is really wrong, so we throw hands up since we exhausted all our options to create the socket. |
@DonJayamanne Does the |
There are no *.sock files in there. |
What about another user data dir? Does that work? Is the one you're using in a special file system (eg. network drive)? |
This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines. Happy Coding! |
Version: 1.39.0-insider
Commit: 601e3e0
Date: 2019-09-30T05:30:22.799Z
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:
Each argument is in a separate line for brevity
'<vs code dir>/Visual Studio Code - Insiders.app' --user-data-dir='<user dir>'
Does this issue occur when all extensions are disabled?: N/A
The text was updated successfully, but these errors were encountered: