2008/5/1 Matt Benson <[EMAIL PROTECTED]>:
> First, thanks to all those who checked and found
>  issues with RC1!
>
>  The artifacts are here:
>  http://people.apache.org/~mbenson/jxpath-1.3-rc2/
>

It would be nice if the Javadoc and Source MANIFEST files contained a
bit more detail.
e.g. Implementation Version, Vendor etc.

The binary manifest says that the code is designed for Java 1.3, yet
it was built and tested in Java 1.5.

M2 requires only Java 1.4, so unless there is any reason why that
cannot be used, at least it would be closer to the target version.
This would at least catch any accidental usage of Java 1.5+ methods.

Ant build fails with Java 1.3.1:

    [junit] Testsuite: org.apache.commons.jxpath.ri.StressTest
    [junit] Tests run: 1, Failures: 0, Errors: 1, Time elapsed: 1.078 sec
    [junit] Tests run: 1, Failures: 0, Errors: 1, Time elapsed: 1.078 sec
    [junit]
    [junit] Testcase: testThreads took 0.688 sec
    [junit]     Caused an ERROR
    [junit] null
    [junit] java.lang.NoSuchMethodError
    [junit]     at
org.apache.commons.jxpath.ri.StressTest$StressRunnable.run(StressTest.java:81)
    [junit]     at java.lang.Thread.run(Thread.java:479)
    [junit]

Also, the Ant build refers to junit-3.8.2 - this is not guaranteed to
work with Java 1.3 as there's a assert method that requires Java 1.4
in it.
This will cause a test failure - if it is used. [There's a posting
about this in the archives earlier this year.]
I don't think this is the cause of the above error.

>  The tag is here:
>  http://svn.apache.org/viewvc/commons/proper/jxpath/tags/JXPATH_1_3_RC2/
>

The rat.xml file is set up to only check Java files.
It should really check all the files apart from specific exceptions -
e.g. the JavaCC-generated files.
However, there don't appear to be any problems.

A few SVN property problems:

svn pd svn:executable
src/java/org/apache/commons/jxpath/util/JXPath11CompatibleTypeConverter.java
svn pd svn:executable
src/java/org/apache/commons/jxpath/util/ReverseComparator.java
svn pd svn:executable
src/test/org/apache/commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java

svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathAbstractFactoryException.java
svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathFunctionNotFoundException.java
svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathInvalidAccessException.java
svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathInvalidSyntaxException.java
svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathNotFoundException.java
svn ps svn:eol-style native
src/java/org/apache/commons/jxpath/JXPathTypeConversionException.java

Not sure about this one; check what the other JSPs have:

svn ps svn:eol-style native src/test-webapp/JXPathTest.jsp

These are not blockers.
>  Site:
>  http://people.apache.org/~mbenson/jxpath-1.3-rc2/site
>
>  Clirr Report (compared to 1.2; one-shot not working w/
>  M2)
>  http://people.apache.org/~mbenson/jxpath-1.3-rc2/clirr-report.txt
>
>  I'd be grateful if you can make time to check the
>  artifacts and cast your vote, which will be open at
>  least until Friday, May 9.
>
>  Thanks,
>  Matt
>
>
>
>       
> ____________________________________________________________________________________
>  Be a better friend, newshound, and
>  know-it-all with Yahoo! Mobile.  Try it now.  
> http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: [EMAIL PROTECTED]
>  For additional commands, e-mail: [EMAIL PROTECTED]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to