Maven plugin (spring-boot:run) should take list for JVM arguments

Problem: It is too easy to break spring-boot:run by passing in garbage in the <jvmArguments> configuration. Currently it looks like this (not a list):

<plugin>
  <groupId>org.springframework.boot</groupId>
  <artifactId>spring-boot-maven-plugin</artifactId>
  <configuration>
    <jvmArguments>any kind of garbage</jvmArguments>
  </configuration>
</plugin>

Solution: The Maven plugin should allow passing in Java (JVM) system properties via some kind of list definition in XML.

Example solution:

<plugin>
  <groupId>org.springframework.boot</groupId>
  <artifactId>spring-boot-maven-plugin</artifactId>
  <configuration>
    <jvmArguments>
      <options>
        <option>-Xms512m</option>
        <option>-Xmx1024m</option>
      </options>
      <systemProperties>
        <systemProperty>systemProperty1=value1</systemProperty>
        <systemProperty>systemProperty2=value2</systemProperty>
      </systemProperties>
    </jvmArguments>
  </configuration>
</plugin>

where each <option> is passed in as-is to the JVM, and each <systemProperty> is pre-pended with a -D:

-Xms512m -Xmx1024m -DsystemProperty1=value1 -DsystemProperty2=value2

[alan-czajkowski]

@vadeg i’m not a fan of “listing” elements like that in XML, my preference is that element lists should be clean looking whereby the top-level element is the plural word of what the list is, and each element is the singular word, example:

<apples>
  <apple>red</apple>
  <apple>green</apple>
  <apple>yellow</apple>
</apples>

even Spring uses this approach! https://www.tutorialspoint.com/spring/spring_injecting_collection.htm

[snicoll]

Closing in favour of PR #10741