Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.1, 2.1.1
    • Fix Version/s: 2.0.3, 2.1, 2.1.1, 2.2
    • Component/s: webservices
    • Security Level: public (Regular issues)
    • Labels:
      None

      Description

      The code in WsdlGenerator is only looking at the current configuration when constructing a classpath to feed to wsgen. If you have an ear with a ws in a war that uses classes in the ear lib or an ejb jar this won't be enough. I think the solution is to include all the ancestors....

      1. GERONIMO-3831.patch
        3 kB
        David Jencks
      2. GERONIMO-3831-2.patch
        5 kB
        David Jencks
      3. GERONIMO-3831-2a.patch
        5 kB
        David Jencks
      4. GERONIMO-3831-3.patch
        11 kB
        David Jencks
      5. GERONIMO-3831-branches-2.0.patch
        3 kB
        Kevan Miller

        Activity

        Hide
        David Jencks added a comment -

        Proposed fix.

        Show
        David Jencks added a comment - Proposed fix.
        Hide
        Kevan Miller added a comment -

        equivalent patch for branches/2.0 – builds. that's all i know about it...

        Show
        Kevan Miller added a comment - equivalent patch for branches/2.0 – builds. that's all i know about it...
        Hide
        David Jencks added a comment -

        This patch works but needs a lot of cleanup before being applied.

        Show
        David Jencks added a comment - This patch works but needs a lot of cleanup before being applied.
        Hide
        David Jencks added a comment -

        same patch as -2 but from the plugins/jaxws/geronimo-jaxws-builder directory for easier application to earlier geronimo versions.

        Show
        David Jencks added a comment - same patch as -2 but from the plugins/jaxws/geronimo-jaxws-builder directory for easier application to earlier geronimo versions.
        Hide
        David Jencks added a comment -

        Fixed in trunk rev 619786, branches 2.1 rev 619787. Looking into 2.0.3...

        Show
        David Jencks added a comment - Fixed in trunk rev 619786, branches 2.1 rev 619787. Looking into 2.0.3...
        Hide
        David Jencks added a comment -

        Patch for branches/2.0. Compiles but otherwise untested.

        Show
        David Jencks added a comment - Patch for branches/2.0. Compiles but otherwise untested.
        Hide
        David Jencks added a comment -

        Merged to branches/2.0 (took a little work) in rev 619795.

        Show
        David Jencks added a comment - Merged to branches/2.0 (took a little work) in rev 619795.
        Hide
        Jarek Gawor added a comment -

        Looks like some of these changes broke jaxws-tools command line tool.

        jgawor@homer:~/target> geronimo-tomcat6-jee5-2.0.3-SNAPSHOT/bin/jaxws-tools.sh wsgen
        Using GERONIMO_BASE: /home/jgawor/target/geronimo-tomcat6-jee5-2.0.3-SNAPSHOT
        Using GERONIMO_HOME: /home/jgawor/target/geronimo-tomcat6-jee5-2.0.3-SNAPSHOT
        Using GERONIMO_TMPDIR: var/temp
        Using JRE_HOME: /home/jgawor/software/jdk1.5.0_12/jre
        Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/geronimo/common/DeploymentException
        at org.apache.geronimo.jaxws.builder.JAXWSToolsCLI.main(JAXWSToolsCLI.java:50)

        Show
        Jarek Gawor added a comment - Looks like some of these changes broke jaxws-tools command line tool. jgawor@homer:~/target> geronimo-tomcat6-jee5-2.0.3-SNAPSHOT/bin/jaxws-tools.sh wsgen Using GERONIMO_BASE: /home/jgawor/target/geronimo-tomcat6-jee5-2.0.3-SNAPSHOT Using GERONIMO_HOME: /home/jgawor/target/geronimo-tomcat6-jee5-2.0.3-SNAPSHOT Using GERONIMO_TMPDIR: var/temp Using JRE_HOME: /home/jgawor/software/jdk1.5.0_12/jre Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/geronimo/common/DeploymentException at org.apache.geronimo.jaxws.builder.JAXWSToolsCLI.main(JAXWSToolsCLI.java:50)
        Hide
        Jarek Gawor added a comment -

        Removed the geronimo-common class dependency from JAXWSTools.java. Changes committed to trunk (revision 631688), branches/2.1 (revision 631689), and branches/2.0 (revision 631690).

        Show
        Jarek Gawor added a comment - Removed the geronimo-common class dependency from JAXWSTools.java. Changes committed to trunk (revision 631688), branches/2.1 (revision 631689), and branches/2.0 (revision 631690).

          People

          • Assignee:
            Jarek Gawor
            Reporter:
            David Jencks
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development