On 5/16/07, Antonio Petrelli <[EMAIL PROTECTED]> wrote:
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?

They are, but usually copied and pasted into a text file and shipped
with the distro - no?

Niall

Thanks for the comments!
Antonio

Reply via email to