github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

Workaround to avoid installing npm when creating a new project on Windows (Fixes #11040)

Browse Source
This commit is contained in:
filipenevola
2021-03-05 08:48:36 -04:00
parent d6842e3972
commit f7ea6a52a4
1 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
tools/cli/commands.js
View File

@@ -837,12 +837,15 @@ main.registerCommand({
// the packages (or maybe an unpredictable subset based on what happens to be // the packages (or maybe an unpredictable subset based on what happens to be
// in the template's versions file). // in the template's versions file).
// Since some of the project skeletons include npm `devDependencies`, we need const isWindows = process.platform === "win32";
// to make sure they're included when running `npm install`. if (!isWindows) {
require("./default-npm-deps.js").install( // Since some of the project skeletons include npm `devDependencies`, we need
appPath, // to make sure they're included when running `npm install`.
{ includeDevDependencies: true } require("./default-npm-deps.js").install(
); appPath,
{includeDevDependencies: true}
);
}
var appNameToDisplay = appPathAsEntered === "." ? var appNameToDisplay = appPathAsEntered === "." ?
"current directory" : `'${appPathAsEntered}'`; "current directory" : `'${appPathAsEntered}'`;
@@ -873,6 +876,10 @@ main.registerCommand({
cmd("cd " + appPathWithQuotesIfSpaces); cmd("cd " + appPathWithQuotesIfSpaces);
} }
if (isWindows) {
cmd("meteor npm install");
}
cmd("meteor"); cmd("meteor");
Console.info(""); Console.info("");