-
Notifications
You must be signed in to change notification settings - Fork 29.6k
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
win: add customization warning to tools script #24348
win: add customization warning to tools script #24348
Conversation
I'm not against clear text, but its effectiveness is limited, especially IMO when it's long. Ohh there's a GUI to create WinForms - https://poshgui.com https://theitbros.com/powershell-gui-for-scripts/ |
Please upvote this comment to approve fast-tracking, it would be good if this could make it into v11.2.0. cc @refack (as requested in collaborator guide) CI: https://ci.nodejs.org/view/All/job/node-test-pull-request/18598/ |
Need another upvote. @nodejs/platform-windows @tniessen @nodejs/build-files |
/CC @nodejs/documentation |
Resume Build CI: https://ci.nodejs.org/view/All/job/node-test-pull-request/18607/ |
Resumed CI green, landed in 3856d8a. Thanks! |
PR-URL: #24348 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
LTS: Depends on #24344, for now it looks like the revert is going forward so I'll add the don't land label here. |
PR-URL: #24348 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Notable changes: * deps: * A new experimental HTTP parser (`llhttp`) is now supported. #24059 * timers: * Fixed an issue that could cause setTimeout to stop working as expected. #24322 * Windows * A crashing process will now show the names of stack frames if the node.pdb file is available. #23822 * Continued effort to improve the installer's new stage that installs native build tools. #23987, #24348 * child_process: * On Windows the `windowsHide` option default was restored to `false`. This means `detached` child processes and GUI apps will once again start in a new window. #24034 * Added new collaborators: * [oyyd](https://github.com/oyyd) - Ouyang Yadong. #24300 * [psmarshall](https://github.com/psmarshall) - Peter Marshall. #24170 * [shisama](https://github.com/shisama) - Masashi Hirano. #24136
Notable changes: * deps: * A new experimental HTTP parser (`llhttp`) is now supported. #24059 * timers: * Fixed an issue that could cause setTimeout to stop working as expected. #24322 * Windows * A crashing process will now show the names of stack frames if the node.pdb file is available. #23822 * Continued effort to improve the installer's new stage that installs native build tools. #23987, #24348 * child_process: * On Windows the `windowsHide` option default was restored to `false`. This means `detached` child processes and GUI apps will once again start in a new window. #24034 * Added new collaborators: * [oyyd](https://github.com/oyyd) - Ouyang Yadong. #24300 * [psmarshall](https://github.com/psmarshall) - Peter Marshall. #24170 * [shisama](https://github.com/shisama) - Masashi Hirano. #24136 PR-URL: #24350
The script introduced in #22645 to install Visual Studio and Python on Windows should work reliably in most scenarios. But it can only go so far, if a user wants to customize the installation or has custom policies in place, the alternative installation methods should be used. The script is designed to be as simple as possible, because for other options and customization there are already detailed instructions that the users can follow.
This PR adds a warning about that.
cc @nodejs/platform-windows
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes