Skip to content

Commit 5796254

Browse files
authored
Merge pull request #14708 from apache/fix-web-profile-tooling-api
Additional outstanding issues for grails-shell-cli
2 parents 9fef7c0 + f0961a1 commit 5796254

File tree

5 files changed

+9
-12
lines changed

5 files changed

+9
-12
lines changed

gradle/assemble-config.gradle

+1-2
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,13 @@
1919

2020
tasks.register('installToHomeDist').configure { Task task ->
2121
task.inputs.files(tasks.named('jar'))
22-
task.inputs.files(tasks.named('sourcesJar'))
23-
task.inputs.files(tasks.named('javadocJar'))
2422
task.outputs.dir(distInstallDir)
2523

2624
doLast {
2725
copy {
2826
it.from layout.buildDirectory.dir('libs')
2927
it.into distInstallDir
28+
it.exclude '*sources.jar', '*-javadoc.jar'
3029
}
3130
}
3231
}

gradle/assemble-root-config.gradle

+1-2
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,7 @@ tasks.register('assemble', Zip).configure {
116116

117117
into("grails-$grailsVersion") {
118118
from(rootProject.layout.projectDirectory) {
119-
// Some of these are probably not needed as they are not present in the project folder
120-
include 'bin/grails', 'bin/grails.bat', 'lib/', 'media/', 'samples/', 'scripts/', 'LICENSE', 'INSTALL'
119+
include 'bin/grails', 'bin/grails.bat', 'lib/', 'media/icons/grails.icns', 'LICENSE', 'INSTALL', 'NOTICE'
121120

122121
def pathsToExclude = [
123122
'ant/bin', 'src/grails', 'src/war'

grails-shell-cli/src/main/groovy/org/grails/cli/GrailsCli.groovy

+1-6
Original file line numberDiff line numberDiff line change
@@ -507,12 +507,7 @@ class GrailsCli {
507507
private initializeProfile() {
508508
BuildSettings.TARGET_DIR?.mkdirs()
509509

510-
if(!new File(BuildSettings.BASE_DIR, "profile.yml").exists()) {
511-
populateContextLoader()
512-
}
513-
else {
514-
this.profileRepository = createMavenProfileRepository()
515-
}
510+
this.profileRepository = createMavenProfileRepository()
516511

517512
String profileName = applicationConfig.get(BuildSettings.PROFILE) ?: getSetting(BuildSettings.PROFILE, String, DEFAULT_PROFILE_NAME)
518513
this.profile = profileRepository.getProfile(profileName)

grails-shell-cli/src/main/groovy/org/grails/cli/boot/GrailsDependencyVersions.groovy

+2
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,8 @@ class GrailsDependencyVersions implements DependencyManagement {
7676
grape.addResolver([name:"mavenCentral", root:"https://repo1.maven.org/maven2"] as Map<String, Object>)
7777
}
7878

79+
grape.addResolver([name:"grailsCentral", root:"https://repo.grails.org/grails/core"] as Map<String, Object>)
80+
7981
grape
8082
}
8183

grails-wrapper-impl/src/main/groovy/grails/init/RunCommand.groovy

+4-2
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,13 @@ class RunCommand {
5151

5252
GroovyClassLoader groovyClassLoader = new GroovyClassLoader(RunCommand.classLoader)
5353

54-
List<RepositoryConfiguration> repositoryConfigurations = [new RepositoryConfiguration("mavenCentral", new URI("https://repo1.maven.org/maven2"), true)]
54+
List<RepositoryConfiguration> repositoryConfigurations = [new RepositoryConfiguration("mavenCentral", new URI("https://repo1.maven.org/maven2"), false)]
55+
56+
repositoryConfigurations.add(new RepositoryConfiguration("grailsCentral", new URI("https://repo.grails.org/grails/core"), true))
5557

5658
// Only add snapshot repository when grailsVersion is not set or groovyVersion or grailsVersion ends in SNAPSHOT
5759
if (!grailsVersion || grailsVersion.endsWith("SNAPSHOT") || groovyVersion?.endsWith("SNAPSHOT")) {
58-
repositoryConfigurations.add(new RepositoryConfiguration("apacheSnapshot", new URI("https://repository.apache.org/content/groups/snapshots"), true))
60+
repositoryConfigurations.add(new RepositoryConfiguration("apacheRepository", new URI("https://repository.apache.org/content/groups/public"), true))
5961
}
6062

6163
MavenResolverGrapeEngine grapeEngine = MavenResolverGrapeEngineFactory.create(groovyClassLoader, repositoryConfigurations, new DependencyResolutionContext(), false)

0 commit comments

Comments
 (0)