Skip to content

Commit 726888d

Browse files
committed
U get default toolchain from javapluginextension only if gradle version is 7 or higher
1 parent 07510e8 commit 726888d

File tree

4 files changed

+53
-53
lines changed

4 files changed

+53
-53
lines changed

src/main/java/io/github/fvarrui/javapackager/gradle/DefaultPackageTask.java

Lines changed: 41 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -15,43 +15,47 @@ protected Packager createPackager() throws Exception {
1515

1616
PackagePluginExtension extension = getProject().getExtensions().findByType(PackagePluginExtension.class);
1717

18-
return (Packager) PackagerFactory.createPackager(extension.getPlatform())
19-
.additionalModules(extension.getAdditionalModules())
20-
.additionalResources(extension.getAdditionalResources())
21-
.administratorRequired(extension.getAdministratorRequired())
22-
.version(defaultIfNull(extension.getVersion(), getProject().getVersion().toString()))
23-
.packagingJdk(extension.getPackagingJdk())
24-
.assetsDir(extension.getAssetsDir())
25-
.bundleJre(extension.getBundleJre())
26-
.copyDependencies(extension.getCopyDependencies())
27-
.createTarball(extension.getCreateTarball())
28-
.createZipball(extension.getCreateZipball())
29-
.customizedJre(extension.getCustomizedJre())
30-
.description(extension.getDescription())
31-
.displayName(extension.getDisplayName())
32-
.envPath(extension.getEnvPath())
33-
.extra(extension.getExtra())
34-
.generateInstaller(extension.getGenerateInstaller())
35-
.iconFile(extension.getIconFile())
36-
.jdkPath(extension.getJdkPath())
37-
.jreDirectoryName(extension.getJreDirectoryName())
38-
.jreMinVersion(extension.getJreMinVersion())
39-
.jrePath(extension.getJrePath())
40-
.licenseFile(extension.getLicenseFile())
41-
.linuxConfig(extension.getLinuxConfig())
42-
.macConfig(extension.getMacConfig())
43-
.mainClass(extension.getMainClass())
44-
.modules(extension.getModules())
45-
.name(extension.getName())
46-
.organizationEmail(extension.getOrganizationEmail())
47-
.organizationName(extension.getOrganizationName())
48-
.organizationUrl(extension.getOrganizationUrl())
49-
.outputDirectory(extension.getOutputDirectory())
50-
.runnableJar(extension.getRunnableJar())
51-
.useResourcesAsWorkingDir(extension.isUseResourcesAsWorkingDir())
52-
.url(extension.getUrl())
53-
.vmArgs(extension.getVmArgs())
54-
.winConfig(extension.getWinConfig());
18+
return
19+
(Packager) PackagerFactory
20+
.createPackager(extension.getPlatform())
21+
.additionalModules(extension.getAdditionalModules())
22+
.additionalModulePaths(extension.getAdditionalModulePaths())
23+
.additionalResources(extension.getAdditionalResources())
24+
.administratorRequired(extension.getAdministratorRequired())
25+
.assetsDir(extension.getAssetsDir())
26+
.bundleJre(extension.getBundleJre())
27+
.copyDependencies(extension.getCopyDependencies())
28+
.createTarball(extension.getCreateTarball())
29+
.createZipball(extension.getCreateZipball())
30+
.customizedJre(extension.getCustomizedJre())
31+
.description(extension.getDescription())
32+
.displayName(extension.getDisplayName())
33+
.envPath(extension.getEnvPath())
34+
.extra(extension.getExtra())
35+
.fileAssociations(extension.getFileAssociations())
36+
.generateInstaller(extension.getGenerateInstaller())
37+
.jdkPath(extension.getJdkPath())
38+
.jreDirectoryName(extension.getJreDirectoryName())
39+
.jreMinVersion(extension.getJreMinVersion())
40+
.jrePath(extension.getJrePath())
41+
.licenseFile(extension.getLicenseFile())
42+
.linuxConfig(extension.getLinuxConfig())
43+
.macConfig(extension.getMacConfig())
44+
.mainClass(extension.getMainClass())
45+
.manifest(extension.getManifest())
46+
.modules(extension.getModules())
47+
.name(extension.getName())
48+
.organizationEmail(extension.getOrganizationEmail())
49+
.organizationName(extension.getOrganizationName())
50+
.organizationUrl(extension.getOrganizationUrl())
51+
.outputDirectory(extension.getOutputDirectory())
52+
.packagingJdk(extension.getPackagingJdk())
53+
.runnableJar(extension.getRunnableJar())
54+
.useResourcesAsWorkingDir(extension.isUseResourcesAsWorkingDir())
55+
.url(extension.getUrl())
56+
.version(defaultIfNull(extension.getVersion(), getProject().getVersion().toString()))
57+
.vmArgs(extension.getVmArgs())
58+
.winConfig(extension.getWinConfig());
5559

5660
}
5761

src/main/java/io/github/fvarrui/javapackager/gradle/GradleContext.java

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,13 +86,21 @@ public Launch4jLibraryTask getLibraryTask() {
8686
public void setLibraryTask(Launch4jLibraryTask libraryTask) {
8787
this.libraryTask = libraryTask;
8888
}
89-
89+
9090
/**
9191
* Returns project's default toolchain
9292
*
9393
* @return Default toolchain
9494
*/
9595
public File getDefaultToolchain() {
96+
if (project.getGradle().getGradleVersion().compareTo("7") >= 0)
97+
return getToolchain();
98+
else
99+
return super.getDefaultToolchain();
100+
}
101+
102+
private File getToolchain() {
103+
96104
// Default toolchain
97105
JavaToolchainSpec toolchain = project.getExtensions().getByType(JavaPluginExtension.class).getToolchain();
98106

@@ -104,6 +112,7 @@ public File getDefaultToolchain() {
104112
return defaultLauncher.get().getMetadata().getInstallationPath().getAsFile();
105113
}
106114
return super.getDefaultToolchain();
115+
107116
}
108117

109118
}

src/main/java/io/github/fvarrui/javapackager/gradle/PackageTask.java

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -226,18 +226,6 @@ public void setGenerateInstaller(Boolean generateInstaller) {
226226
this.generateInstaller = generateInstaller;
227227
}
228228

229-
@InputFile
230-
@Optional
231-
private File iconFile;
232-
233-
public File getIconFile() {
234-
return iconFile;
235-
}
236-
237-
public void setIconFile(File iconFile) {
238-
this.iconFile = iconFile;
239-
}
240-
241229
@InputDirectory
242230
@Optional
243231
private File jdkPath;
@@ -590,7 +578,6 @@ protected Packager createPackager() throws Exception {
590578
.extra(defaultIfNull(extra, extension.getExtra()))
591579
.fileAssociations(defaultIfNull(fileAssociations, extension.getFileAssociations()))
592580
.generateInstaller(defaultIfNull(generateInstaller, extension.getGenerateInstaller()))
593-
.iconFile(defaultIfNull(iconFile, extension.getIconFile()))
594581
.jdkPath(defaultIfNull(jdkPath, extension.getJdkPath()))
595582
.jreDirectoryName(defaultIfBlank(jreDirectoryName, extension.getJreDirectoryName()))
596583
.jreMinVersion(defaultIfBlank(jreMinVersion, extension.getJreMinVersion()))
@@ -606,7 +593,7 @@ protected Packager createPackager() throws Exception {
606593
.organizationName(defaultIfNull(organizationName, extension.getOrganizationName()))
607594
.organizationUrl(defaultIfNull(organizationUrl, extension.getOrganizationUrl()))
608595
.outputDirectory(defaultIfNull(outputDirectory, extension.getOutputDirectory()))
609-
.packagingJdk(defaultIfNull(packagingJdk, extension.getPackagingJdk(), Context.getContext().getDefaultToolchain()))
596+
.packagingJdk(defaultIfNull(packagingJdk, extension.getPackagingJdk(), Context.getGradleContext().getDefaultToolchain()))
610597
.runnableJar(defaultIfNull(runnableJar, extension.getRunnableJar()))
611598
.scripts(defaultIfNull(scripts, extension.getScripts()))
612599
.useResourcesAsWorkingDir(defaultIfNull(useResourcesAsWorkingDir, extension.isUseResourcesAsWorkingDir()))

src/main/java/io/github/fvarrui/javapackager/packagers/BundleJre.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ protected File doApply(Packager packager) throws Exception {
4545
List<String> additionalModules = packager.getAdditionalModules();
4646
List<File> additionalModulePaths = packager.getAdditionalModulePaths();
4747

48-
File currentJdk = packager.packagingJdk;
48+
File currentJdk = packager.getPackagingJdk();
4949

5050
Logger.infoIndent("Bundling JRE ... with " + currentJdk);
5151

0 commit comments

Comments
 (0)