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 “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

Release 6.11.1 #236

Closed
wants to merge 1,157 commits into from
Closed

Release 6.11.1 #236

wants to merge 1,157 commits into from

Conversation

syedbilal07
Copy link

Hey, i don't know where to open a solution for issue in latest npm (6.11.1) but i've found a solution for
"'find_dp0' is not recognized as an internal or external command"
and
" Cannot find module 'C:\node_modules\npm\bin\npm-cli.js'"

Please edit npm.cmd in C:\Program Files\nodejs and paste the below npm.cmd code so you can use npm again.

@echo off
SETLOCAL

IF EXIST "%dp0%\node.exe" (
SET "_prog=%dp0%\node.exe"
) ELSE (
SET "_prog=node"
SET PATHEXT=%PATHEXT:;.JS;=;%
)

"%_prog%" "%dp0%\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" %*
ENDLOCAL
EXIT /b
find_dp0:
SET dp0=%~dp0
EXIT /b

zkat and others added 30 commits January 23, 2019 14:07
Make sure publishing with legacy username:password _auth works again
isaacs and others added 24 commits August 15, 2019 10:34
Prevent root-owned files in user-owned locations, and vice versa.
Don't use SUDO_UID and SUDO_GID.  Just make the file match the folder
it's being written into.  The vast majority of the time, this will be
the user's home directory, but if it's not, then we should not leave a
user-owned file in a root-owned location.  And, if running as root
without SUDO_UID/SUDO_GID environs, but putting a config file in the
user's home dir, then it's quite rude to leave it root-owned.
Do not leave a root-owned package-lock or npm-shrinkwrap file in the
project root, where it will create problems when the user tries to
update it later.
This seems to break every time a dependency is updated.  Not sure why
node 6 is any different than any other environment, but it's probably
not worth debugging any further.
PR-URL: #235
Credit: @theberbie
Close: #235
Reviewed-by: @isaacs

EDIT(isaacs): Added test
@isaacs
Copy link
Contributor

isaacs commented Aug 28, 2019

This is already fixed in 6.11.2, I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet