+1 ... with just one potential concern:

While not officially part of the binary distribution, I did notice that the build/management scripts in the tag root source don't include the AL header. I'm never sure just how paranoid we need to be about the AL header for marginal parts like these. Given that these are only used to create the image, I don't think it's a real concern (hence my +1). However, I thought I should mention it just in case somebody has some justification for why a header must be on those parts too. I personally think it's probably safest just to add the header for the next release.

Joe


David Jencks wrote:
For various reasons we are now using a forked copy of tomcat derived from the tomcat 6.0.20 release, built with maven, with maven dependencies, etc etc.

The code changes from tomcat 6.0.20 are:

r814083 | djencks | 2009-09-11 16:36:16 -0700 (Fri, 11 Sep 2009) | 1 line

GERONIMO-4566 port fix for tomcats https://issues.apache.org/bugzilla/show_bug.cgi?id=47378
------------------------------------------------------------------------
r799220 | xuhaihong | 2009-07-30 02:53:30 -0700 (Thu, 30 Jul 2009) | 1 line

GERONIMO-3451 "Restricted listeners property file not found" error logged during Tomcat server startup (Patch from Shawn Jiang)
------------------------------------------------------------------------
r790767 | kevan | 2009-07-02 17:09:12 -0700 (Thu, 02 Jul 2009) | 1 line

GERONIMO-4685 Some customizations of Tomcat properties. Server info is now: Apache Geronimo (Embedded Tomcat). Will want to merge in the future.
------------------------------------------------------------------------
r790765 | kevan | 2009-07-02 17:06:02 -0700 (Thu, 02 Jul 2009) | 1 line

GERONIMO-4685 Merge of 783305 from tomcat/tc6.0.x/trunk/. Fixes NPE in MapperListener.registerHost(). I would expect that this fix would be in 6.0.22. So prolly won't require merging in the future
------------------------------------------------------------------------
r790742 | kevan | 2009-07-02 14:41:08 -0700 (Thu, 02 Jul 2009) | 1 line

GERONIMO-4685 Merge of Tomcat patches from 6.0.18 to 6.0.20. This includes revision numbers 784303 and 784304. Does not include 787519, which should already be in 6.0.20
------------------------------------------------------------------------
r790359 | gawor | 2009-07-01 13:23:00 -0700 (Wed, 01 Jul 2009) | 1 line

fill-in ServerInfo properties - but I can't add that to archetype as ant task will fail

Staging repo:

https://repository.apache.org/content/repositories/geronimo-staging-023/

I forgot to include site generation stuff in the root pom before rolling the release. I generated a site by applying rev 815573 to the checkout used for the release build. The results are here:

http://people.apache.org/~djencks/staging-site/maven/external/tomcat-parent-6.0.20/6.0.20.0/

The structure is a little weird but I don't really want to spend time fixing it up.


[ ] +1 go for it
[ ] 0
[ ] -1 whoa, hold on a minute

Vote open 72 hours

thanks
david jencks




Reply via email to