Skip to content

Commit

Permalink
Issue #6476 - Show message if JVM args are present but new JVM is spa…
Browse files Browse the repository at this point in the history
…wned

Updates after review: reduced the WARN message lines.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
  • Loading branch information
sbordet committed Aug 11, 2021
1 parent ef05630 commit 054b7c7
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions jetty-start/src/main/java/org/eclipse/jetty/start/Main.java
Expand Up @@ -475,9 +475,7 @@ else if (args.isCreateFiles() || !args.getStartModules().isEmpty())
.filter(module -> !module.getJvmArgs().isEmpty())
.map(Module::getName)
.collect(Collectors.toList());
StartLog.warn("Forking second JVM due to forking module(s): %s.", execModules);
StartLog.warn("Consider using option --dry-run to generate the command line:");
StartLog.warn("$(java -jar $JETTY_HOME/start.jar --dry-run)");
StartLog.warn("Forking second JVM due to forking module(s): %s. Use --dry-run to generate the command line to avoid forking.", execModules);

ProcessBuilder pbuilder = new ProcessBuilder(cmd.getArgs());
StartLog.endStartLog();
Expand Down

0 comments on commit 054b7c7

Please sign in to comment.