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

Broken "Ensure Prelaunch Dependencies" task #106877

Closed
roblourens opened this issue Sep 16, 2020 · 11 comments
Closed

Broken "Ensure Prelaunch Dependencies" task #106877

roblourens opened this issue Sep 16, 2020 · 11 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority insiders-released Patch has been released in VS Code Insiders tasks Task system issues verified Verification succeeded

Comments

@roblourens
Copy link
Member

Can't use the Launch VS Code config because the prelaunch task is broken. The terminal process failed to launch: Starting directory (cwd) "/Users/roblou/code/vscode/~/code/vscode" does not exist.

@connor4312 @alexr00 any idea? I don't see what's determining the terminal cwd.

@connor4312
Copy link
Member

connor4312 commented Sep 16, 2020

I had an old Insiders on my macbook, it worked there and then failed after updating. Looking into, have a feeling it's tasks-related though as I haven't touched this in several weeks. The prelaunch task is as simple as it gets

		{
			"type": "shell",
			"command": "node build/lib/preLaunch.js",
			"label": "Ensure Prelaunch Dependencies",
			"presentation": {
				"reveal": "silent"
			}
		},

@connor4312
Copy link
Member

connor4312 commented Sep 16, 2020

Seems to have been caused by 49ec999, I suspect by moving some of the path.normalize to path.relative. I don't have enough context to safely fix it myself.

Marking important since this seems to affect all shell tasks on OSX

@connor4312 connor4312 assigned alexr00 and unassigned connor4312 Sep 16, 2020
@connor4312 connor4312 added the important Issue identified as high-priority label Sep 16, 2020
@weinand weinand added bug Issue identified by VS Code Team member as probable bug tasks Task system issues labels Sep 16, 2020
@alexr00
Copy link
Member

alexr00 commented Sep 17, 2020

To support remote properly we should never be calling path.normalize like that, so I removed it. Looks like somewhere it's required though...

@alexr00
Copy link
Member

alexr00 commented Sep 17, 2020

Turned out to be unrelated to the path.normalize, and instead it was a missing parameter to the label service call.

@mastrauckas
Copy link

@alexr00 When will this be released? My VSCode is completely broken.

@OoLunar
Copy link

OoLunar commented Sep 17, 2020

This issue also affects Ubuntu 20.04.1 LTS. VSCode version: 1.50.0-insider \d1f267742169584db010716d92addf50d4cd8139
Guess this is a wakeup call to finally finish my documentation...

@payonel
Copy link

payonel commented Sep 17, 2020

@OoLunar what documentation would that be, and how would it help us in this situation? just curious :)
also, I believe i have reported this issue in a duplicate ticket here: #106958
I'll close my ticket myself when i can pull the updated build and retest

@OoLunar
Copy link

OoLunar commented Sep 17, 2020

@OoLunar what documentation would that be, and how would it help us in this situation? just curious :)

@payonel Oh it wouldn't help anyone here in the slightest. I've just been procrastinating creating docs for my project for months now... That's offtopic though 😅

@OoLunar
Copy link

OoLunar commented Sep 18, 2020

It's fixed! Least, on the Debian ecosystem. Thank y'all for the quick fix!

@LoeDK
Copy link

LoeDK commented Sep 18, 2020

Sadly, this just fixed one aspect of the bug for me.
Executing tasks now has the cwd set as ~/... instead of /home/user/... leading to errors within scripts.
The behaviour used to be /home/user... .
The bug introduced /home/user/~/... .
The "bugfix" introduced ~/... as default, which poses problems

System is Ubuntu 18.04

@LoeDK
Copy link

LoeDK commented Sep 18, 2020

I opened a new ticket for this issue:
#106988 106988

@roblourens roblourens added the verified Verification succeeded label Sep 30, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Nov 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority insiders-released Patch has been released in VS Code Insiders tasks Task system issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

9 participants