Skip to content

Commit 133bbe4

Browse files
authored
Merge pull request #337 from orange-guo/topic/xcg/error_msg
improvement: Improve the error message when downloading the AppImageTool fails
2 parents 00ccc83 + e1c45b6 commit 133bbe4

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

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

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,14 @@
11
package io.github.fvarrui.javapackager.packagers;
22

3-
import java.io.File;
4-
import java.io.IOException;
5-
6-
import org.apache.commons.lang3.SystemUtils;
7-
83
import io.github.fvarrui.javapackager.model.Platform;
94
import io.github.fvarrui.javapackager.utils.CommandUtils;
105
import io.github.fvarrui.javapackager.utils.FileUtils;
116
import io.github.fvarrui.javapackager.utils.Logger;
127
import io.github.fvarrui.javapackager.utils.VelocityUtils;
8+
import org.apache.commons.lang3.SystemUtils;
9+
10+
import java.io.File;
11+
import java.io.IOException;
1312

1413
public class GenerateAppImage extends ArtifactGenerator<LinuxPackager> {
1514

@@ -97,7 +96,7 @@ private File getAppImageTool(LinuxPackager packager) throws Exception {
9796
try {
9897
FileUtils.downloadFromUrl(imageToolUrl, appImageTool);
9998
} catch (IOException e) {
100-
throw new Exception(imageToolUrl + "not found! ... Unsupported OS architecture " + getOSArch() + "?");
99+
throw new Exception("An error occurred while downloading appimagetool from " + imageToolUrl + " for " + getOSArch() + "! It may be a network problem or the url " + imageToolUrl + " is not valid!", e);
101100
}
102101
appImageTool.setExecutable(true);
103102
}

0 commit comments

Comments
 (0)