X Tutup
The Wayback Machine - https://web.archive.org/web/20221005045113/https://github.com/nodejs/node/pull/43236
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 鈥淪ign up for GitHub鈥, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: fix Visual Studio 2019 download link #43236

Merged
merged 1 commit into from Jun 21, 2022

Conversation

F3n67u
Copy link
Member

@F3n67u F3n67u commented May 29, 2022

Background: Building Node.js with Visual Studio 2022 is broken from Visual Studio 17.2, see #43092.

The https://visualstudio.microsoft.com/downloads/ now link to the latest Visual Studio 2022 version instead of 2019 is incorrect. If our windows new contributor follows the link and downloads the Visual Studio 2022, build will fail. I follow this link and download the 2022 version yesterday馃槀 and experience build failure.

https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2019 link is also broken, I cannot find a link to build-tools-for-visual-studio-2019 to replace it, so I delete the [Build Tools] link. If #43092 is resolved, we could add it again. or If anyone could find the link to build-tools-for-visual-studio-2019, please comment and I will add it.

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. labels May 29, 2022
the "C++ build tools" workload from the
[Build Tools](https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2019),
[Build Tools](https://aka.ms/vs/16/release/vs_buildtools.exe),
Copy link
Member Author

@F3n67u F3n67u May 29, 2022

Choose a reason for hiding this comment

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

https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2019 link is also broken, I cannot find a visual studio 2019 c++ build tools link from https://visualstudio.microsoft.com/downloads.

Using https://aka.ms/vs/16/release/vs_buildtools.exe link to download visual studio 2019 build tools is documented at
https://docs.microsoft.com/en-us/visualstudio/install/build-tools-container?view=vs-2019#create-and-build-the-dockerfile.

image

I have verified this link does works on my windows server: uninstall already installed visual studio 2019, install build tool via the build tools link, and verify .\vcbuild does compile.

@F3n67u F3n67u requested a review from targos Jun 11, 2022
@F3n67u
Copy link
Member Author

F3n67u commented Jun 19, 2022

@targos could you please take a look at this pr?

targos
targos approved these changes Jun 19, 2022
@F3n67u
Copy link
Member Author

F3n67u commented Jun 21, 2022

Hi @targo, could you help to land this pr? Thanks.

@targos targos added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 21, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 21, 2022
@nodejs-github-bot nodejs-github-bot merged commit 08d6a82 into nodejs:main Jun 21, 2022
15 checks passed
@nodejs-github-bot
Copy link
Contributor

nodejs-github-bot commented Jun 21, 2022

Landed in 08d6a82

@F3n67u F3n67u deleted the fix/vs2019-link branch Jun 21, 2022
codebytere pushed a commit to codebytere/node that referenced this pull request Jun 21, 2022
PR-URL: nodejs#43236
Reviewed-By: Micha毛l Zasso <targos@protonmail.com>
F3n67u added a commit to F3n67u/node that referenced this pull request Jun 24, 2022
PR-URL: nodejs#43236
Reviewed-By: Micha毛l Zasso <targos@protonmail.com>
targos pushed a commit that referenced this pull request Jul 12, 2022
PR-URL: #43236
Reviewed-By: Micha毛l Zasso <targos@protonmail.com>
targos pushed a commit that referenced this pull request Jul 20, 2022
PR-URL: #43236
Reviewed-By: Micha毛l Zasso <targos@protonmail.com>
targos pushed a commit that referenced this pull request Jul 31, 2022
PR-URL: #43236
Reviewed-By: Micha毛l Zasso <targos@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants
X Tutup