Skip to content

Commit

Permalink
Fixes #3898, log an info instead of o warning when entrypoint makes t… (
Browse files Browse the repository at this point in the history
#3904)

* Fixes #3898, log an info instead of warning when entrypoint makes the image to ignore jvm parameters
  • Loading branch information
rmannibucau committed Jan 9, 2023
1 parent 456ec0b commit a4a7f54
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -596,7 +596,7 @@ static List<String> computeEntrypoint(
|| !rawExtraClasspath.isEmpty()
|| rawConfiguration.getExpandClasspathDependencies())) {
projectProperties.log(
LogEvent.warn(
LogEvent.info(
"mainClass, extraClasspath, jvmFlags, and expandClasspathDependencies are ignored "
+ "when entrypoint is specified"));
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -642,7 +642,7 @@ public void testEntrypoint_warningOnJvmFlags()
assertThat(buildPlan.getEntrypoint()).containsExactly("custom", "entrypoint").inOrder();
verify(projectProperties)
.log(
LogEvent.warn(
LogEvent.info(
"mainClass, extraClasspath, jvmFlags, and expandClasspathDependencies are ignored "
+ "when entrypoint is specified"));
}
Expand All @@ -664,7 +664,7 @@ public void testEntrypoint_warningOnMainclass()
assertThat(buildPlan.getEntrypoint()).containsExactly("custom", "entrypoint").inOrder();
verify(projectProperties)
.log(
LogEvent.warn(
LogEvent.info(
"mainClass, extraClasspath, jvmFlags, and expandClasspathDependencies are ignored "
+ "when entrypoint is specified"));
}
Expand All @@ -686,7 +686,7 @@ public void testEntrypoint_warningOnExpandClasspathDependencies()
assertThat(buildPlan.getEntrypoint()).containsExactly("custom", "entrypoint").inOrder();
verify(projectProperties)
.log(
LogEvent.warn(
LogEvent.info(
"mainClass, extraClasspath, jvmFlags, and expandClasspathDependencies are ignored "
+ "when entrypoint is specified"));
}
Expand Down

0 comments on commit a4a7f54

Please sign in to comment.