pombreda / gwt-maven

Automatically exported from code.google.com/p/gwt-maven
0 stars 1 forks source link

Multiple modules don't compile on Solaris or Linux #211

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Is this Maven 1 or Maven 2?
Maven 2

What steps will reproduce the problem?
1. Create a multiple module project, specify the modules in the pom
<compileTargets>
    <value>com.mymodules.ModuleA</value>
    <value>com.mymodules.ModuleB</value>
</compileTargets>
2. Do a "mvn package" on Solaris - fails
3. Do a "mvn package" on Windows - passes

What is the expected output? 
Compiling module com.mymodules.ModuleA
   Compiling 5 permutations
   Compilation succeeded
Compiling module com.mymodules.ModuleB
   Compiling 5 permutations
   Compilation succeeded

What do you see instead?
Loading module 'com.mymodules.ModuleAcom.mymodules.ModuleB'
   [ERROR] Unable to 
find 'com/mymodules/ModuleAcom/mymodules/ModuleB.gwt.xml' on your 
classpath; could be a typo, or maybe you forgot to include a classpath 
entry for source?

What version of the product are you using? On what operating system?
Solaris + Linux, 2.0-RC1, GWT 1.6.4, 

Please provide any additional information below.
Looks like a compiler problem on linux, it appears to be mashing the two 
modules into one.

The reason I am "packaging" on Solaris is this is where my CruiseControl 
is located.

Original issue reported on code.google.com by knight...@gmail.com on 3 Jul 2009 at 2:54

GoogleCodeExporter commented 9 years ago
I've located the bug and fixed locally, can someone in GWT confirm and 
patch "com\totsp\gwt\maven-googlewebtoolkit2-plugin\2.0-RC1"
Offending file is
http://code.google.com/p/gwt-maven/source/browse/trunk/maven-googlewebtoolkit2-
plugin/src/main/java/com/totsp/mavenplugin/gwt/scripting/ScriptWriterUnix16.java

line number 191

code should be
        for (String target : mojo.getCompileTarget()) { 
             writer.print(target + " "); 
         } 

Original comment by knight...@gmail.com on 3 Jul 2009 at 6:24