Skip to content

This will look for Visual Studio in the Correct directory. - #143479

Merged
Robo (deepak1556) merged 14 commits into
mainfrom
unknown repository
Feb 28, 2022
Merged

Robo (deepak1556) merged 14 commits into
mainfrom
unknown repository

Conversation

@ghost

Copy link
Copy Markdown

Previously running yarn install was looking for visual studio workload in the programfilesx86 folder now this fix will check for visual studio in the programfilesx86 in the first and then will look on programFiles folder.

This fix will help Visual Studio 2022 users as it is a 64b bit software so it goes to programFiles folder not in ProgramFilesx86.

This PR fixes #143478

@ghost Deleted user (ghost) left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have checked the code and it works in my case

@samarmohan

Copy link
Copy Markdown

I've tried this before, and the preinstall.js code works fine, but Electron and node-gyp don't like the 64 bit version.
There was an issue regarding this a while ago but I can't find it.

@samarmohan

Copy link
Copy Markdown

Ah well, I was behind on this issue. #138572 and #142667

@ghost

Copy link
Copy Markdown
Author

Yaa this will actually fix the issue and the next time no other will have the same issue. Just waiting for the maintainer to merge this pr

@ghost ghost changed the title This is look for Visual Studio in the Correct directory. This will look for Visual Studio in the Correct directory. Feb 20, 2022
@ghost

Copy link
Copy Markdown
Author

I tested the code and it worked totally fine. Let's see when the maintainer merges this pr.

@ghost

Copy link
Copy Markdown
Author

After this fix the issue will completely resolve.

@ghost

Copy link
Copy Markdown
Author

Paula (@paulacamargo25) please approve all the pending workflow.

@ghost

Copy link
Copy Markdown
Author

Paula (@paulacamargo25) if there's something wrong in the code please let me know.

Comment thread build/npm/preinstall.js Outdated
@ghost

Copy link
Copy Markdown
Author

Raymond Zhao (@rzhao271) waiting for your approval.

@rzhao271

Copy link
Copy Markdown
Collaborator

It is currently endgame/testing week. PR LGTM, but I plan on merging it next week.

@rzhao271 Raymond Zhao (rzhao271) added this to the March 2022 milestone Feb 23, 2022
@rzhao271 Raymond Zhao (rzhao271) added the engineering VS Code - Build / issue tracking / etc. label Feb 23, 2022
@ghost

Copy link
Copy Markdown
Author

No issues. Waiting for the endgame to end.

@deepak1556

Copy link
Copy Markdown
Collaborator

Merging to address build failure https://github.com/microsoft/vscode/runs/5352655839?check_suite_focus=true

@deepak1556

Copy link
Copy Markdown
Collaborator

Thanks for the PR!

@deepak1556
Robo (deepak1556) merged commit d35ee52 into microsoft:main Feb 28, 2022
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

engineering VS Code - Build / issue tracking / etc.

Projects

None yet

4 participants