Skip to content

Commit ab90049

Browse files
committed
Merge pull request #418 from NativeScript/plamen5kov/remove_npm_install_from_build
update gradle to remove npm install from it
2 parents 0264513 + 9555e95 commit ab90049

File tree

2 files changed

+1
-42
lines changed

2 files changed

+1
-42
lines changed

build/project-template-gradle/build-tools/android-static-binding-generator/ast-parser/package.json

Lines changed: 0 additions & 18 deletions
This file was deleted.

build/project-template-gradle/build-tools/android-static-binding-generator/build.gradle

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -44,28 +44,6 @@ task generateInterfaceNamesList() {
4444
}
4545
}
4646

47-
// won't run if node_modules are installed
48-
task runNpmInstallForAstParser () {
49-
50-
//check this way so it doesn't slow down the build by snapshot-ing the node_modules
51-
outputs.upToDateWhen {
52-
(new File("$astParserDir/node_modules")).exists()
53-
}
54-
55-
doFirst {
56-
exec {
57-
workingDir astParserDir
58-
59-
if(isWinOs) {
60-
commandLine "cmd", "/c", "npm", "install"
61-
}
62-
else {
63-
commandLine "npm", "install"
64-
}
65-
}
66-
}
67-
}
68-
6947
// if there are new dependencies the parser will run again
7048
task runAstParser () {
7149
inputs.files fileTree(dir: absoluteJsCodeDir)
@@ -108,7 +86,6 @@ task generateBindings() {
10886
}
10987
}
11088

111-
runNpmInstallForAstParser.dependsOn(generateInterfaceNamesList)
112-
runAstParser.dependsOn(runNpmInstallForAstParser)
89+
runAstParser.dependsOn(generateInterfaceNamesList)
11390
generateBindings.dependsOn(runAstParser)
11491

0 commit comments

Comments
 (0)