Issue Details (XML | Word | Printable)

Key: XALANJ-2159
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Henry Zongaro
Reporter: Henry Zongaro
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
XalanJ2

Class path and bootstrap class path are not set correctly for test harness in Gump build

Created: 28/Jun/05 11:38 AM   Updated: 19/Aug/05 10:48 AM
Return to search
Component/s: TestHarness-TestCase
Affects Version/s: 2.7.1
Fix Version/s: 2.7

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works patch.j2159.txt 2005-06-28 11:40 AM Henry Zongaro 1 kB

Xalan info: PatchAvailable
Reviewer: Joanne Tong
Resolution Date: 30/Jun/05 05:53 AM


 Description  « Hide
The nightly Gump build is failing on the xml-xalan2-smoketest "project." This is because of interactions between the way Gump sets up the class path and bootstrap class path for invoking ant and how the xml-xalan/test/build.xml file puts together the class path and bootstrap class path for the xalantest task.

In particular, Gump places xml-apis.jar, xercesImpl.jar and xalan.jar on the boot class path. The build.xml, however, takes the class path and specifies that as the bootstrap class path for the xalantest task. It also adds the xercesImpl.jar and xml-apis.jar files that can be found in xml-xalan/java/bin to the bootstrap class path, so that it doesn't actually perform integration testing, which is what Gump was intended to do.

The problem can be resolved by copying through the class path and bootstrap class path settings for the smoketest.gump target.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Joanne Tong added a comment - 29/Jun/05 12:58 AM
I reviewed this patch and have no comments at this time. Since this problem exists in a specific environment, please commit this patch and if it fixes the gump failure, then I find this patch acceptable to resolve this bug.

Henry Zongaro added a comment - 30/Jun/05 05:53 AM
Applied patch in CVS. It appears to resolve the Gump build failure.

I learned from Stefan Bodewig on the Gump general mailing list[1] of a CommandlineJava.createBootclasspath(Project) method that's part of Ant 1.6. If and when Xalan-J migrates to that version of Ant, the test harness's org.apache.qetest.xsl.XSLTestAntTask.setBootclasspathref method should use that method of CommandlineJava to set the boot class path instead of using createVmArg. Then my work-around in build.xml can be removed.

[1] http://mail-archives.apache.org/mod_mbox/gump-general/200506.mbox/%3cOFA9B6CE08.C71CF3A5-ON8525702F.004B12FE-8525702F.00624C32@ca.ibm.com%3e

Brian Minchau added a comment - 19/Aug/05 10:48 AM
GUMP is no longer broken because of this. Closing this issue.