Here is the patch and a snippet of the maven test:

[junit] Running org.apache.commons.jelly.core.TestInvokeStaticTag
[junit] Tests run: 5, Failures: 0, Errors: 0, Time elapsed: 3.896 sec
[junit] Running org.apache.commons.jelly.core.TestInvokeTag
[junit] Tests run: 6, Failures: 0, Errors: 0, Time elapsed: 3.146 sec

Note that I'm not sure if the tags.html has been updated correctly - I couldn't run maven site:generate

Test cases fail after applying the patch.

I'll investigate

The patch was missing changes to testInvokeStaticTag.jelly and others

Hmm, let me create a new patch then...

Here is the new patch.

Sorry for the missing files - I didn't do a recursive patch because of JELLY-115 (i.e., my cvs repository has 2 issues patched)

Will wait for the new patch before trying again.

Also, when you throw the JellyException, can you please use the constructor that takes a string and a Throwable so the original exception details are preserved?

Thanks.

Fixed with minor changes

Here is the change you've requested - preserving the original exception.

This patch appears to be a JDK 1.4 specific one.

It uses Throwable.getCause() which was introduced in jdk 1.4

I'm commenting those lines out for now.

Sorry, my bad, I used the wrong casts.

Yes, Throwable.getCause() is JDK 1.4, but I'm using InvocationTarget.getTargetMessage() instead:

http://java.sun.com/j2se/1.3/docs/api/java/lang/reflect/InvocationTargetException.html

I mean, there is also a call to jellyException.getCause(), but JellyException defines it indenpendently on the JDK version.

Anyway, I've just ran maven on it using JDK 1.3.1, and it worked fine. It's just a matter of modifying the test cases, changing the following line:

from:
Exception jellyException = (Exception) getJellyContext().getVariable("jellyException");

to:
JellyException jellyException = (JellyException) getJellyContext().getVariable("jellyException");

And also adding a

import org.apache.commons.jelly.JellyException;

(Sorry, I'm not with CVS set up right now to send you a patch)

Fixing now