2007/5/16, Niall Pemberton <[EMAIL PROTECTED]>:

Running the tests causes the following warning along with a stack trace
WARNING: Cannot find TilesContextFactory class
org.apache.tiles.jsp.context.JspTilesContextFactory

Seeing a stack trace makes me think theres a problem that needs
resolving - better IMO to log the warning without the stack trace.



Ok, I'll manage to change the log messages.


I also saw a load of messages in the output referring to the
apache.snapshots
(http://people.apache.org/repo/m2-snapshot-repository) repository -
looking at the pom the repository section has a comment saying "Needs
to be commented out for releases".



In fact we're not relying on snapshots releases anymore, but I think this is
not a problem, since the Maven release plugin automatically checks snapshot
dependencies, and if they are found, the release cannot be done.


Minor nitpick - the tiles-core and tiles-jsp jars have duplicate
LICENSE and NOTICE files (in root and meta-inf directories).



You're right, the tiles-api pom defines the exclusion of LICENSE and NOTICE
from the root, this rule should be applied to all poms.


RAT report is clean :)



A clean RAT is not an utopia, after all :-)


Personally I would prefer to see the release notes in the distribution
- rather than a Jira link.


I think that posting JIRA release notes is pretty standard. Or am I missing
something?

Thanks for the comments!
Antonio

Reply via email to