Issue 62422 - Update of Apache Xalan-J implementation from 2.6.0 to 2.7.0
Summary: Update of Apache Xalan-J implementation from 2.6.0 to 2.7.0
Status: CLOSED FIXED
Alias: None
Product: xml
Classification: Code
Component: external filters (show other issues)
Version: OOo 2.0.2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.0.4
Assignee: jogi
QA Contact: issues@xml
URL:
Keywords:
: 64170 (view as issue list)
Depends on:
Blocks:
 
Reported: 2006-02-22 11:20 UTC by svante.schubert
Modified: 2006-09-25 10:22 UTC (History)
2 users (show)

See Also:
Issue Type: ENHANCEMENT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
full workspace (3.83 KB, patch)
2006-07-24 11:23 UTC, caolanm
no flags Details | Diff
XSLT stylesheet to get the vendor info from Xalan (984 bytes, text/xml)
2006-08-02 16:39 UTC, jogi
no flags Details
first.bas test result (20.46 KB, application/octet-stream)
2006-08-03 09:18 UTC, jogi
no flags Details
topten.bas test result (20.77 KB, application/octet-stream)
2006-08-03 09:20 UTC, jogi
no flags Details
Writer Update test result (380.47 KB, application/octet-stream)
2006-08-03 09:22 UTC, jogi
no flags Details
ODF Graphics test result (17.05 KB, application/octet-stream)
2006-08-03 09:24 UTC, jogi
no flags Details
DocBook (XSLT filter) test result (57.82 KB, application/octet-stream)
2006-08-03 09:25 UTC, jogi
no flags Details
Spreadsheet Import test result (19.59 KB, application/octet-stream)
2006-08-03 09:27 UTC, jogi
no flags Details
Writer XML import test result (66.24 KB, application/octet-stream)
2006-08-03 09:28 UTC, jogi
no flags Details
Test Case Specification for detecting the Xalan used from OOo (6.05 KB, text/html)
2006-08-03 09:31 UTC, jogi
no flags Details
FIX: Type in testvase: 1 "1.0" in changed to "1" (6.04 KB, text/html)
2006-09-25 10:20 UTC, jogi
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description svante.schubert 2006-02-22 11:20:04 UTC
The exisiting Xalan version in OpenOffice.org/StarOffice should be updated from
2.6 to 2.7. It was stated in from various developers, that the 2.7 version is in
a very good
condition.

Furthermore, they added JAXP 1.3 and Namespace/XML 1.1

http://xml.apache.org/xalan-j/readme.html#done

The downside:
Apache splitted the serializer.jar from xalan.jar, therefore all classpathes
have to be adapted.

PS: This change has been approved by project management for StarOffice and
OpenOffice.org.

SUS->CMC:
Hi Caloan, could you please take over this enhancement as we discussed earlier?
You might join my latex CWS, but on the other hand it might be a good idea as
well, to isoloate this task in a single CWS. It's up to you.
One of us should write a change request/announcement about the version change
and JAR change before integrating this CWS.
TIA
Svante
Comment 1 caolanm 2006-02-22 11:31:19 UTC
seperate workspace I think, should depend on post jaxpapi being integated to
avoid getting in a tangle with that
Comment 2 caolanm 2006-04-11 12:14:19 UTC
*** Issue 64170 has been marked as a duplicate of this issue. ***
Comment 3 caolanm 2006-04-26 17:04:37 UTC
done in xalanupgrade
Comment 4 caolanm 2006-04-27 08:13:50 UTC
reopen to reassign for qaing
Comment 5 caolanm 2006-04-27 08:14:35 UTC
not sure who should be qaing this
Comment 6 caolanm 2006-04-27 08:14:59 UTC
done in xalanupgrade
Comment 7 svante.schubert 2006-06-13 16:16:04 UTC
SUS->CMC:
Changed the target milestone to pp4.

I will countertest this issue and we should use Jörg Sievers as QA representive

BTW what have done, what have you changed? Have you added only the JARS or had
you parsed all soucres for e.g. "alan.jar" and added a new "serializer.jar" to
the classpathes as a new JAR has been added for Xalan7?
Comment 8 svante.schubert 2006-07-03 16:56:29 UTC
SUS->CMC:
Any information regarding testing (see above)?
Comment 9 caolanm 2006-07-03 17:32:15 UTC
yeah, Class-Path entries in the various MANIFEST.MF have been updated to include
serializer.jar so all should work out of the box. Unfortunately due to
http://ooomisc.services.openoffice.org/pub/OpenOffice.org/cws/upload/readme.txt
I believe the install sets may have disappeared in the interim
Comment 10 svante.schubert 2006-07-24 11:09:25 UTC
Some things I would likt to mention concerning our XALANUPGRADE CWS:

   1. Is it possible to provide us some links to the install sets for testing? ;-)
   2. Is it possible to do an 'cwsview' on the CWS to find out the changes?

      The main change to the earlier used Xalan 2.6 implementation is that a new
serializer.jar has been splitted from the Xalan jar
      My test approach would be:
           find . -type f | xargs egrep -i "xalan.jar"

      With this I found in a m174 the following files:

      ./filter/source/config/fragments/makefile.mk:
      ./filter/source/xsltfilter/Manifest:
      ./filter/source/xsltfilter/makefile.mk:
      ./filter/source/xsltvalidate/makefile.mk:
      ./scp2/source/ooo/makefile.mk:
      ./scp2/source/ooo/profileitem_ooo.scp:
      ./xmerge/java/build.xml:
      ./xmerge/java/org/openoffice/xmerge/converter/xml/xslt/build.xml:

   3. I realized, that the choice of the preselection of the added Xalan
processor by Java property - I meant to have it earlier added long ago - , is no
longer available in the sources. (find . -type f | xargs egrep -i
"javax.xml.transform.TransformerFactory", did not result something useful).
Therefore I recommend to add a new enhancement to this CWS as well as it has
exact the same test effort/approach.
      Concerning the implementation, I suggest to use a generic preseclection by
Java property in "filter/source/xsltfilter/XSLTransformer.java" with
'javax.xml.transform.TransformerFactory=org.apache.xalan.processor.TransformerFactoryImpl'
 as default.
   4. What about posting the upgrade benefits as JAXP 1.3 and XML 1.1 support
(see http://xml.apache.org/xalan-j/readme.html#done).
      It is not really a feature mail as we don't test the new features, any
suggestions?
Comment 11 caolanm 2006-07-24 11:22:21 UTC
On Wed, 2006-07-19 at 14:39 +0200, Svante Schubert wrote:
> Hi Caolán,
> 
> Some things I would like to discuss with you concerning our
> XALANUPGRADE CWS:
>      1. Is it possible to provide us some links to the install sets
>         for testing? ;-)

http://ooo.services.openoffice.org/pub/OpenOffice.org/cws/upload/xalanupgrade/

>      1. Is it possible to do an 'cwsview' on the CWS to find out the
>         changes?

attached is the full workspace patch (ignoring the changed jar
binaries), it's quite small

>         
>         The main change to the earlier used Xalan 2.6 implementation
>         is that a new serializer.jar has been splitted from the Xalan
>         jar 
>         My test approach would be: 
>              find . -type f | xargs egrep -i "xalan.jar"
>         
>         With this I found in a m174 the following files:
>         
>         ./filter/source/config/fragments/makefile.mk:
>         ./filter/source/xsltfilter/Manifest:
>         ./filter/source/xsltfilter/makefile.mk:
>         ./filter/source/xsltvalidate/makefile.mk:
>         ./scp2/source/ooo/makefile.mk:
>         ./scp2/source/ooo/profileitem_ooo.scp:
>         ./xmerge/java/build.xml:
>         ./xmerge/java/org/openoffice/xmerge/converter/xml/xslt/build.xml:

Note the xalan jars which are users of seralizer.jar mention
seralizer.jar in their Class-Path in the upsteam jars which this
workspace adds, so it definitely needs to be added to scp2, but doesn't
need to be added to every CLASSPATH in the source tree because if the
user jar has the Class-Path entry mentioning the serializer.jar then it
gets found that way.


>      1. I realized, that the choice of the preselection of the added
>         Xalan processor by Java property - I meant to have it earlier
>         added long ago - , is no longer available in the sources.
>         (find . -type f | xargs egrep -i
>         "javax.xml.transform.TransformerFactory", did not result
>         something useful). Therefore I recommend to add a new
>         enhancement to this CWS as well as it has exact the same test
>         effort/approach.
>         Concerning the implementation, I suggest to use a generic
>         preseclection by Java property in
>         "filter/source/xsltfilter/XSLTransformer.java" with
>        
'javax.xml.transform.TransformerFactory=org.apache.xalan.processor.TransformerFactoryImpl'
 as default. 


Now, I'm afraid that with fedora's move to gcc 4, it's not possible
anymore for me to create any workspaces which are able to pass automated
Hamburg QA, so I can't actually build any QA-suitable workspaces. So I
suggest you take ownership of the workspace, as you can create modern
workspaces which are qa-able.
Comment 12 caolanm 2006-07-24 11:23:23 UTC
Created attachment 37982 [details]
full workspace
Comment 13 svante.schubert 2006-07-24 18:40:40 UTC
SUS->JSI:
Could you please verifiy this issue?
The patch looks sufficient, but I havn't installed myself the cws office patch
to review it, in a running Office.
Comment 14 jogi 2006-07-25 09:23:54 UTC
JSI -> CMC/SUS: Please write an Feature Announcement (yes, it is an enhancement
but we all know that it is not unimportant to know for XSLT developers what
we're doing). 

JSI->CMC: It would be great next time to get also second platform installation
set to test. We all know that there are many differences between e.g. Win32 and
Unix/UNIX :-)
Comment 15 caolanm 2006-07-25 11:06:28 UTC
cmc->jsi: yeah, I know. But there is a fundamental problem :-), we don't have
the windows development tools.
Comment 16 svante.schubert 2006-07-26 11:54:09 UTC
Going to take it over, Caolan! 
Thanks for your help so far. 

Just started to create local environment for this CWS here in Hamburg.
Tomorrow we should have install sets for Windows and Linux..

Cheers,
Svante
Comment 17 svante.schubert 2006-07-31 10:55:05 UTC
Added the new and exchanged JARS to the StarOffice patch as well.
Doing this by changing macro of the file_ooo.scp of the scp2 module from
STD_JAR_FILE 
to 
STD_JAR_FILE_PATCH.
Comment 18 jogi 2006-08-02 16:18:11 UTC
verified in CWS; testcase in progress; will br attached to this issue asap.
Comment 19 jogi 2006-08-02 16:39:23 UTC
Created attachment 38217 [details]
XSLT stylesheet to get the vendor info from Xalan
Comment 20 jogi 2006-08-03 09:18:37 UTC
Created attachment 38231 [details]
first.bas test result
Comment 21 jogi 2006-08-03 09:20:13 UTC
Created attachment 38232 [details]
topten.bas test result
Comment 22 jogi 2006-08-03 09:22:07 UTC
Created attachment 38233 [details]
Writer Update test result
Comment 23 jogi 2006-08-03 09:24:35 UTC
Created attachment 38234 [details]
ODF Graphics test result
Comment 24 jogi 2006-08-03 09:25:58 UTC
Created attachment 38235 [details]
DocBook (XSLT filter) test result
Comment 25 jogi 2006-08-03 09:27:19 UTC
Created attachment 38236 [details]
Spreadsheet Import test result
Comment 26 jogi 2006-08-03 09:28:36 UTC
Created attachment 38237 [details]
Writer XML import test result
Comment 27 jogi 2006-08-03 09:31:58 UTC
Created attachment 38238 [details]
Test Case Specification for detecting the Xalan used from OOo
Comment 28 jogi 2006-09-25 10:20:39 UTC
Created attachment 39351 [details]
FIX: Type in testvase: 1 "1.0" in changed to "1"
Comment 29 jogi 2006-09-25 10:22:56 UTC
Verified and fixed testcase. OK in OOD680m4