saaj
  1. saaj
  2. SAAJ-7

Ref Impl does not work with IBM JDK

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Won't Fix
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: www
    • Labels:
      None
    • Environment:

      Operating System: Linux
      Platform: All

      Description

      The 1.3 reference implementation of SAAJ does not work with the IBM JDK.
      Looking at the source code, you see things like:

      factory = new com.sun.org.apache.xerces.internal.jaxp.SAXParserFactoryImpl();

      import com.sun.org.apache.xerces.internal.dom.DocumentImpl;

      extends com.sun.org.apache.xerces.internal.dom.TextImpl

      etc....

      The com.sun.org.apache stuff does not exist in the IBM JDK. Thus, the
      reference SAAJ impl is not "write once, run anywhere".

        Activity

        Hide
        kumarjayanti added a comment -

        Just made a fix by adding an ant target "remove-rename"

        When invoked, this would rename all the dependencies in SAAJ 1.3 RI such that
        the RI can compile with xercesImpl.jar and xalan.jar classes obtained from :

        1. http://xml.apache.org/xerces2-j/download.cgi
        2. http://xml.apache.org/xalan-j/downloads.html

        To be able to run the "remove-rename" target successfully use the following steps

        1. checkout saaj 1.3 workspace
        2. download and place xercesImpl.jar, xalan.jar, serializer.jar, xml-apis.jar
        and xsltc.jar inside saaj-ri/lib
        3. invoke "ant remove-rename"

        This will create a directory "remove-rename" under saaj-ri that will contain
        the entire set of saaj sources renamed to use the Apache Xercers Implementation.

        There is a sanity test that happens upon invoking remove-rename which will try
        to compile all the classes and place them under the directory
        saaj-ri/build/nonrenamed-classes

        You can then jar these classes into a saaj-impl.jar or use the compiled classes
        as necessary.

        Please use this feature and let us know if this worked. If you are satisfied
        with the fix then you can probably close the bug or downgrade the priority.

        Thanks,
        kumar

        Show
        kumarjayanti added a comment - Just made a fix by adding an ant target "remove-rename" When invoked, this would rename all the dependencies in SAAJ 1.3 RI such that the RI can compile with xercesImpl.jar and xalan.jar classes obtained from : 1. http://xml.apache.org/xerces2-j/download.cgi 2. http://xml.apache.org/xalan-j/downloads.html To be able to run the "remove-rename" target successfully use the following steps 1. checkout saaj 1.3 workspace 2. download and place xercesImpl.jar, xalan.jar, serializer.jar, xml-apis.jar and xsltc.jar inside saaj-ri/lib 3. invoke "ant remove-rename" This will create a directory "remove-rename" under saaj-ri that will contain the entire set of saaj sources renamed to use the Apache Xercers Implementation. There is a sanity test that happens upon invoking remove-rename which will try to compile all the classes and place them under the directory saaj-ri/build/nonrenamed-classes You can then jar these classes into a saaj-impl.jar or use the compiled classes as necessary. Please use this feature and let us know if this worked. If you are satisfied with the fix then you can probably close the bug or downgrade the priority. Thanks, kumar
        Hide
        kumarjayanti added a comment -

        Waived for WSIT milestone 3 since it does not affect WSIT in anyway.

        Show
        kumarjayanti added a comment - Waived for WSIT milestone 3 since it does not affect WSIT in anyway.
        Hide
        kumarjayanti added a comment -

        We do not have the bandwidth to fix this issue at this point.

        Show
        kumarjayanti added a comment - We do not have the bandwidth to fix this issue at this point.

          People

          • Assignee:
            saaj-issues
            Reporter:
            dkulp
          • Votes:
            9 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: