Skip to content

Commit 19c1813

Browse files
Gradle plugins version updates (#954)
1 parent 7b7f3cf commit 19c1813

5 files changed

Lines changed: 25 additions & 21 deletions

File tree

build.gradle

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,15 +108,15 @@ allprojects {
108108
if (project.hasProperty("artifactory_contextUrl"))
109109
{
110110
maven {
111-
url "${artifactory_contextUrl}/ext-tools-local"
111+
url = "${artifactory_contextUrl}/ext-tools-local"
112112
content {
113113
includeGroup "org.labkey.tools.windows"
114114
includeGroup "org.labkey.tools.linux"
115115
includeGroup "org.labkey.tools.osx"
116116
}
117117
}
118118
maven {
119-
url "${artifactory_contextUrl}/libs-release-no-proxy"
119+
url = "${artifactory_contextUrl}/libs-release-no-proxy"
120120

121121
if (project.hasProperty('artifactory_user') && project.hasProperty('artifactory_password'))
122122
{
@@ -133,7 +133,7 @@ allprojects {
133133
}
134134
}
135135
maven {
136-
url "${artifactory_contextUrl}/libs-snapshot-no-proxy"
136+
url = "${artifactory_contextUrl}/libs-snapshot-no-proxy"
137137

138138
if (project.hasProperty('artifactory_user') && project.hasProperty('artifactory_password'))
139139
{
@@ -165,7 +165,7 @@ allprojects {
165165
maven {
166166
// Mondrian dependencies are available via this repository. It's a direct dependency of the Query
167167
// module but is declared here as many modules depend on Query and therefore need it as well.
168-
url "https://repo.orl.eng.hitachivantara.com/artifactory/pnt-mvn"
168+
url = "https://repo.orl.eng.hitachivantara.com/artifactory/pnt-mvn"
169169
content {
170170
includeGroup "pentaho"
171171
includeGroup "org.pentaho"
@@ -506,7 +506,7 @@ project.tasks.register('listNpmProjects', DefaultTask) {
506506
List<String> npmProjects = []
507507
project.allprojects({Project p ->
508508
if (p.getPlugins().hasPlugin(NpmRun.class))
509-
npmProjects.add("${p.path} (${NpmRun.useYarn(p) ? 'yarn' : 'npm'})")
509+
npmProjects.add(p.path)
510510
})
511511
if (npmProjects.size() == 0)
512512
println("No projects found containing ${NpmRun.NPM_PROJECT_FILE}")

gradle.properties

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ windowsProteomicsBinariesVersion=1.0
5858

5959
# The current version numbers for the gradle plugins.
6060
artifactoryPluginVersion=5.2.5
61-
gradleNodePluginVersion=3.5.1
62-
gradlePluginsVersion=4.2.0
61+
gradleNodePluginVersion=7.1.0
62+
gradlePluginsVersion=6.0.0
6363
owaspDependencyCheckPluginVersion=11.1.0
6464
versioningPluginVersion=1.1.2
6565

@@ -73,9 +73,6 @@ nodeRepo=https://nodejs.org/dist
7373
nodeWorkDirectory=.node
7474
# Directory in a project's build directory where the npm binary will be placed
7575
npmWorkDirectory=.node
76-
#yarnVersion=1.16.0
77-
# Directory in a project's build directory where the yarn binary will be placed
78-
#yarnWorkDirectory=.node
7976

8077
# uncomment the line below to include version number info in build artifacts
8178
# Generally you don't need or want to do this when developing.

server/build.gradle

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -230,9 +230,16 @@ else
230230
// We add this configuration here so we have a single location to link to for the npm and node executables.
231231
// Each project that requires node will have its own downloaded version of node and npm, but for the symlinkNode
232232
// task we need a single location, and one that works even when not building from source (Issue 35207)
233-
project.node {
234-
workDir = BuildUtils.getRootBuildDirFile(project, ".node")
235-
npmWorkDir = BuildUtils.getRootBuildDirFile(project, ".node")
236-
yarnWorkDir = BuildUtils.getRootBuildDirFile(project, ".node")
233+
if (BuildUtils.useServerNode(project))
234+
{
235+
project.node {
236+
workDir = project.file("${project.rootProject.projectDir}/.node")
237+
npmWorkDir = project.file("${project.rootProject.projectDir}/.node")
238+
}
239+
} else {
240+
project.node {
241+
workDir = BuildUtils.getRootBuildDirFile(project, ".node")
242+
npmWorkDir = BuildUtils.getRootBuildDirFile(project, ".node")
243+
}
237244
}
238245
project.tasks.named('deployApp').configure { dependsOn(project.tasks.npmSetup) }

server/embedded/build.gradle

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,10 @@ project.tasks.register(
9090
"checkVersionConflicts", CheckForVersionConflicts)
9191
{ CheckForVersionConflicts task ->
9292
task.group = GroupNames.BUILD
93-
task.description = "Check for conflicts in version numbers of jar file and files already in the build directory ${ServerDeployExtension.getEmbeddedServerDeployDirectory(project)}. " +
93+
task.description = "Check for conflicts in version numbers of jar file and files already in the build directory ${ServerDeployExtension.getEmbeddedServerDeployDirectoryPath(project)}. " +
9494
"Default action on detecting a conflict is to fail. Use -PversionConflictAction=[delete|fail|warn] to change this behavior. The value 'delete' will cause the " +
95-
"conflicting version(s) in the ${ServerDeployExtension.getEmbeddedServerDeployDirectory(project)} directory to be removed."
96-
task.directory = new File(ServerDeployExtension.getEmbeddedServerDeployDirectory(project))
95+
"conflicting version(s) in the ${ServerDeployExtension.getEmbeddedServerDeployDirectoryPath(project)} directory to be removed."
96+
task.directory = new File(ServerDeployExtension.getEmbeddedServerDeployDirectoryPath(project))
9797
task.extension = "jar"
9898
task.cleanTask = "server:cleanEmbeddedDeploy"
9999
task.collection = project.tasks.bootJar.outputs.files

settings.gradle

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ pluginManagement {
88
}
99
}
1010
maven {
11-
url "${artifactory_contextUrl}/plugins-release-no-proxy"
11+
url = "${artifactory_contextUrl}/plugins-release-no-proxy"
1212
mavenContent {
1313
releasesOnly()
1414
}
@@ -21,7 +21,7 @@ pluginManagement {
2121
{
2222
mavenLocal()
2323
maven {
24-
url "${artifactory_contextUrl}/plugins-snapshot-local"
24+
url = "${artifactory_contextUrl}/plugins-snapshot-local"
2525
mavenContent {
2626
snapshotsOnly()
2727
}
@@ -57,7 +57,7 @@ buildscript {
5757
}
5858
}
5959
maven {
60-
url "${artifactory_contextUrl}/plugins-release"
60+
url = "${artifactory_contextUrl}/plugins-release"
6161
mavenContent {
6262
releasesOnly()
6363
}
@@ -68,7 +68,7 @@ buildscript {
6868
// You will need to rerun the publish task to pick up new changes.
6969
mavenLocal()
7070
maven {
71-
url "${artifactory_contextUrl}/plugins-snapshot-local"
71+
url = "${artifactory_contextUrl}/plugins-snapshot-local"
7272
mavenContent {
7373
snapshotsOnly()
7474
}

0 commit comments

Comments
 (0)