Mojo's GWT Maven Plugin
  1. Mojo's GWT Maven Plugin
  2. MGWT-219

In gwt:compile the gwt-dev library should be on the classpath before the project dependencies

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 2.1.0-1
    • Labels:
      None
    • Number of attachments :
      1

      Description

      When compiling drools-guvnor with gwt:compile I does this:

      ...java -Xmx512m -classpath "/home/ge0ffrey/projects/jboss/drools/drools-guvnor/src/main/java":
      "/home/ge0ffrey/projects/jboss/drools/drools-guvnor/src/main/resources":
      "/home/ge0ffrey/projects/jboss/drools/drools-guvnor/target/classes":
      ...
      "/home/ge0ffrey/.m2/repository/org/eclipse/jdt/core/compiler/ecj/3.5.1/ecj-3.5.1.jar":
      ...
      "/home/ge0ffrey/.m2/repository/com/google/gwt/gwt-dev/2.0.4/gwt-dev-2.0.4.jar"
      com.google.gwt.dev.Compiler ...

      The problem is that both the gwt-dev jar and the jdt-core:ecj jar have the ecj classes in their jar.
      Even worse, gwt-dev 2.0.4 does not work with ecj 3.5.1.
      If the gwt-dev is first on the classpath, the compile would work (now it fails with a NoSuchFieldError).
      Of course if any if the client classes would use jdt, this would be a problem (which isn't the case)...

      See
      http://code.google.com/p/google-web-toolkit/issues/detail?id=5290
      http://code.google.com/p/google-web-toolkit/issues/detail?id=4479

      1. MGWT-219.patch
        0.9 kB
        Geoffrey De Smet

        Activity

        Hide
        Olivier Lamy added a comment -

        fixed rev 13242.
        Mojo attribute : gwtSdkFirstInClasspath (or -Dgwt.gwtSdkFirstInClasspath=true)
        Have Fun !
        And it added

        Show
        Olivier Lamy added a comment - fixed rev 13242. Mojo attribute : gwtSdkFirstInClasspath (or -Dgwt.gwtSdkFirstInClasspath=true) Have Fun ! And it added
        Hide
        Olivier Lamy added a comment -

        @Geoffrey If you could have a look at MGWT-222 that will be nice

        Show
        Olivier Lamy added a comment - @Geoffrey If you could have a look at MGWT-222 that will be nice
        Hide
        Geoffrey De Smet added a comment -

        @Olivier thanks! I 've added testing this and 222 to my short term todo list (about a week backlog atm).

        Show
        Geoffrey De Smet added a comment - @Olivier thanks! I 've added testing this and 222 to my short term todo list (about a week backlog atm).
        Hide
        Olivier Lamy added a comment -

        yes please test this.
        I'd like to push this release in the middle of next week.

        Show
        Olivier Lamy added a comment - yes please test this. I'd like to push this release in the middle of next week.
        Hide
        Geoffrey De Smet added a comment -

        Confirming it's fixed. thank you! Sorry for the delay

        Details of my confirmation:
        Default config fails with the JDT version problem: java.lang.NoSuchFieldError: warningThreshold
        Config with <gwtSdkFirstInClasspath>true</gwtSdkFirstInClasspath> works.

        Show
        Geoffrey De Smet added a comment - Confirming it's fixed. thank you! Sorry for the delay Details of my confirmation: Default config fails with the JDT version problem: java.lang.NoSuchFieldError: warningThreshold Config with <gwtSdkFirstInClasspath>true</gwtSdkFirstInClasspath> works.

          People

          • Assignee:
            Olivier Lamy
            Reporter:
            Geoffrey De Smet
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: