For the tiles-documentation, it will be an issue for the 1.1 final release. I am not sure it is for a b2 release. I will correct it asap, but I don't know when I will find enough time to do it. As again, any help and patches are welcomes ;-).
Cedric Martin Cooper wrote: >The CVS tree was tagged on Saturday night, per the release plan, and I have >a build from that code base ready to be uploaded to the web site. (The CVS >tree for corresponding Commons and Commons Sandbox components was also >tagged.) > >However, in my testing, I uncovered some issues that I wanted to mention >before making the build available, in case any of the committers think we >should hold Beta 2 until they are resolved. So, here's what I found. > >Unit Tests >========== > >The JUnit tests, and tests against Tomcat 4.0.4, passed. However, tests >against Tomcat 3.2.4 failed. The reason appears to be a class loader bug in >that version of Tomcat. Note that the release plan does not call for working >with Tomcat 3.2.4, although we have unit tests for that environment. > >Sample Webapps >============== > >I tested with the following containers: > >* Tomcat versions 3.2.4, 3.3.1, 3.3.2-dev, 4.0.4 >* Resin versions 1.2.10, 2.1.2, 2.1.4 > >Because of the apparent class loader bug noted above, I did little testing >with Tomcat 3.2.4 beyond confirming that the example webapp fails in the >same way as the unit tests. > >In addition, test results for Tomcat 3.3.1 and 3.3.2-dev appeared identical, >as did the results from Resin 2.1.2 and 2.1.4. > >Here are the issues I found. > >All containers >-------------- > >* The struts-blank webapp does not work in any of the tested containers. The >issue is related to message resources not being found. Apparently, this >webapp has two separate resource files, although only one is configured in >the Struts config file. This just needs someone to expend a little TLC to >put it back into working order. > >* The tiles-documentation webapp has several broken links in the main >navigation bar. The includes are failing because they are referencing >non-existent files. (Only Resin provided the information about which files >it was looking for and failing to find.) The four links I found that were >broken were to the tutorial, installation, user guide and Javadocs. > >Tomcat 3.3.1 and 3.3.2-dev >-------------------------- > >* The <html:cookie> test throws an exception because a method being accessed >is not public. Given that all other containers pass this test, I suspect >that this is a bug in these versions of Tomcat, but I have not investigated >this further. > >* The Comparison Tags test fails rather dramatically, with a JVM error >shutting down the container, and complaining about a problematic thread. >This happened both with JDK 1.4.0_01 and with JDK 1.3.1_01. I don't know >whether this is a JVM bug exposed only by Tomcat 3.3.x, or a Tomcat 3.3.x >bug, or something else. It seems unlikely to be a Struts bug. > >All versions of Resin >--------------------- > >* The <html:cookie> test shows "null" in the Correct Value column, while the >Test Result column shows a missing table cell. In contrast, all versions of >Tomcat show a missing table cell in the Correct Value column. I have not >investigated this further. > >Resin 2.1.2 and 2.1.4 >--------------------- > >* The struts-example webapp seems to keep losing the session id, so that any >time you try to edit your registration data, you are taken to the logon page >again, even if you just finished logging on. I suspect that this is a Resin >bug, given some of the recent comments on the Resin mailing list, but it's >possible that this is a tag handler reuse issue with the example webapp. I >have not investigated this further. > >Conclusion >========== > >Although there are some nasty problems described above, many of them appear >to be container specific. Those that are not, such as the bugs in >struts-blank and tiles-documentation, are not, in my opinion, sufficiently >severe to warrant holding up a Beta 2 release, although they certainly need >to be fixed before a 1.1 Final release. > >My recommendation, therefore, is to proceed with the Beta 2 release, based >on the current tagged code base, and to resolve the Struts related issues >described above shortly thereafter. > >Comments? > >-- >Martin Cooper > > >-- >To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> >For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> > > > > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
