[MCOMPILER-279] Passing multiple --add-exports args to the compilerArgs is not supported

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.5.1
• Fix Version/s: 3.6.0
• Assignee: Robert Scholte
• Reporter: Michael Musgrove
• Created: 29/Sep/16 14:48
• Updated: 04/Nov/20 08:29
• Resolved: 30/Sep/16 19:00

Description

This plugin collapses multiple compiler args with the same name into a single one.

An example of where this is an issue is when exporting extra packages from a Java SE 9 module:

Suppose we want to export two packages (package1 and package2) from a module called FROM-MOD to the OTHER-MOD module. The syntax for this is:

javac ... --add-exports FROM-MOD/package1=OTHER-MOD --add-exports FROM-MOD/package2=OTHER-MOD

and the plugin configuration for this is:

<configuration>
  ...
  <compilerArgs>
    <arg>--add-exports</arg>
    <arg>FROM-MOD/package1=OTHER-MOD</arg>
    <arg>--add-exports</arg>
    <arg>FROM-MOD/package2=OTHER-MOD></arg>
  <compilerArgs>
</configuration>

which produces:

javac ... --add-exports FROM-MOD/package1=OTHER-MOD FROM-MOD/package2=OTHER-MOD

Notice the missing --add-exports for the second export so the compiler complains with the error:

Caused by: java.lang.IllegalArgumentException: invalid flag: java.corba/com.sun.corba.se.spi.ior=ALL-UNNAMED

A concrete but basic example maven project is attached.