Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8257539: tools/jpackage/windows/WinL10nTest.java unpack.bat failed wi…
…th Exit code: 1618

Reviewed-by: almatvee, asemenyuk, prr, kizune
  • Loading branch information
Andy Herrick committed Dec 10, 2020
1 parent 53e537c commit 1ce2a36
Showing 1 changed file with 3 additions and 2 deletions.
Expand Up @@ -63,7 +63,7 @@ private static Path getInstallationSubDirectory(JPackageCommand cmd) {

private static void runMsiexecWithRetries(Executor misexec) {
Executor.Result result = null;
for (int attempt = 0; attempt != 3; ++attempt) {
for (int attempt = 0; attempt < 8; ++attempt) {
result = misexec.executeWithoutExitCodeCheck();

// The given Executor may either be of an msiexe command or an
Expand All @@ -72,7 +72,8 @@ private static void runMsiexecWithRetries(Executor misexec) {
if ((result.exitCode == 1618) || (result.exitCode == 1603)) {
// Another installation is already in progress.
// Wait a little and try again.
ThrowingRunnable.toRunnable(() -> Thread.sleep(3000)).run();
Long timeout = 1000L * (attempt + 3); // from 3 to 10 seconds
ThrowingRunnable.toRunnable(() -> Thread.sleep(timeout)).run();
continue;
}
break;
Expand Down

1 comment on commit 1ce2a36

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.