Home Original page

doc: fix copy node executable in Windows by yoavain · Pull Request #48624 · nodejs/node

@yoavain 

Windows where command lists all places it finds a pattern in Path.
The first one is the one that executes when called.
So the old code was overriding the first executable by any other match.

@nodejs-github-bot added the doc

Issues and PRs related to the documentations.

label

Jul 1, 2023

@yoavain 

Found a much easier solution for copying the node executable using native node.
Can probably be consolidated to all OSs.

@yoavain 

Merged command for both PowerShell and Command Prompt

RaisinTen

@RaisinTen RaisinTen added author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

commit-queue-squash

Add this label to instruct the Commit Queue to squash all the PR commits into the first one.

labels

Jul 4, 2023

@lpinca lpinca added the commit-queue

Add this label to land a pull request using GitHub Actions.

label

Jul 6, 2023

juanarbol pushed a commit that referenced this pull request

Jul 13, 2023 
Windows where command lists all places it finds a pattern in Path.
The first one is the one that executes when called.
So the old code was overriding the first executable by any other match.

PR-URL: #48624
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Ceres6 pushed a commit to Ceres6/node that referenced this pull request

Aug 14, 2023 
Windows where command lists all places it finds a pattern in Path.
The first one is the one that executes when called.
So the old code was overriding the first executable by any other match.

PR-URL: nodejs#48624
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Ceres6 pushed a commit to Ceres6/node that referenced this pull request

Aug 14, 2023 
Windows where command lists all places it finds a pattern in Path.
The first one is the one that executes when called.
So the old code was overriding the first executable by any other match.

PR-URL: nodejs#48624
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>