On 14/04/2010 21:06, sebb wrote:
> Many of the java and xml files in tomcat-lite don't have AL headers.
Not an issue. tomcat-lite isn't being shipped (at least it shoudln't be..)

> The following other files don't have AL headers:
> 
> res/config.ini
> res/jvm.ini
> test/org/apache/coyote/http11/TestInternalInputBuffer.java
> test/org/apache/naming/resources/TestBaseDirContext.java
> 
> The java files definitely need AL headers.
Yep. fixed.

> I don't know the syntax of the .ini files, so it may be impossible to
> add comments, but if comments are possible, then the appropriate
> header should be added as these are not trivial files.
I think I have the format for this. Need to build the installer to test it.

> 2 files in BCEL have IBM headers; these headers are presumably OK, but
> the NOTICE file probably needs to mention IBM. Not sure why the BCEL
> source archive does not do so in its NOTICE file. That might be an
> error.
Hmm. I might be able to strip down what we use from BCEL to get rid of
those.

> Otherwise the source archives and SVN tag look OK, and they agree with
> each other apart from .classpath and .project, which aren't needed in
> the archives.
As expected.

> The binary archives generally look good, with N&L files and useful manifests.
> However, the Implementation-version headers include -RC1.
> Not sure that's correct; depends how the RC is promoted to GA.
Current thinking is tag with another vote that should be quick as folks
will just need to diff the approved RC with the new source tarball.

> There is a minor problem with the bin/tomcat-juli.jar - the NOTICE says:
> Copyright 19...@year@
I'll take a look.

> And for some odd reason there is an MD5 hash for it in the directory.
> The file could probably be removed.
And at that.

> The bin/tomcat-native.tar.gz file includes a KEYS file which probably
> does not belong.
Since it can be a separate download, I'm happy with that.

> My vote would be -1 because of the licensing issues.
Yep. RC2 coming up.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to