Age | Commit message (Collapse) | Author |
|
Co-authored-by: Marco Tizzoni <marco.tizzoni@ecb.europa.eu>
|
|
1. Set the shebang to /usr/bin/env bash instead of /bin/sh (which might
be dash or some other shell)
2. Use Unix-style line endings, not Windows-style (Cygwin accepts
either, but mingw bash sometimes objects, and WSL bash always does)
3. Test against paths using wslpath if available, but still pass win32
paths to node.exe, since it is a Windows binary that only knows how
to handle Windows paths.
This makes npm as installed by the Node.js Windows MSI installer behave
properly under WSL, Cygwin, MINGW Git Bash, and the internal MINGW Git
Bash when posix CLI utilities are exposed to the cmd.exe shell.
The test is not quite as comprehensive as I'd like. It runs on the
various Windows bash implementations if they are found in their expected
locations, skipping any that are not installed. Short of shipping
mingw, cygwin, and wsl as test fixtures, I'm not sure how we could do
much better, however. At least, we can use this test to assist debug
and catch issues on Windows machines (ours or users who report
problems).
PR-URL: https://github.com/npm/cli/pull/2789
Credit: @isaacs
Close: #2789
Reviewed-by: @nlf
|
|
|
|
This reverts commit f019a248a67e8c46dbe41bf31f4818c5ca2138bf.
Git Bash on Windows still uses this file. Removing it broke
the windows MSI builds in Node.js infrastructure. Unless we
absolutely need to remove this I think it is worth keeping
around
PR-URL: https://github.com/npm/cli/pull/1903
Credit: @MylesBorins
Close: #1903
Reviewed-by: @isaacs
|
|
Close: https://github.com/npm/cli/pull/1792
|
|
This reverts commit 3471d5200217bfa612b1a262e36c9c043a52eb09.
PR-URL: https://github.com/npm/cli/pull/527
Credit: @craigloewen-msft
Close: #527
Reviewed-by: @mikemimik
|
|
This commit would make it possible to run `npm` installed on Windows within Windows Subsystem for Linux.
It is required, however, that the end of line characters of `npm` and `npx` be `\n` for the fix to take effect.
PR-URL: https://github.com/npm/cli/pull/253
Credit: @infinnie
Close: #253
Reviewed-by: @claudiahdz
|
|
It was incorrectly calling `node npx-cli prefix` which led to downloading "prefix" package.
Instead it should be calling `node npm-cli prefix`
PR-URL: https://github.com/npm/npm/pull/19608
Credit: @laggingreflex
Reviewed-By: @zkat
|
|
npx is an npm package runner with a bunch of nice features. While
it is also available on the npm registry as `npx`, it's the sort
of thing that's better off being bundles with npm itself.
Fixes: #6053
PR-URL: https://github.com/npm/npm/pull/17685
Credit: @zkat
Reviewed-By: @iarna
|