commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matt Benson <>
Subject [VOTE] Release Apache Commons Weaver 1.0 based on RC2
Date Wed, 05 Mar 2014 00:24:13 GMT
Apache Commons Weaver 1.0 RC2 is available for review at: (r4567).

Maven artifacts are at:

Tested with Oracle JDKs 6, 7 and 8.

The Subversion tag is:


RAT Report:


The following issues were raised with RC1:

> - The names of the distributions files contain a -dist suffix, and they
> also deflate in a directory named -dist. This is not typical for other
> commons components.

Fixed; the dist module is now named commons-weaver.

> - The binary distribution does not contain the release notes.


> - The jar artifacts contain LICENSE and NOTICE without the .txt extension.


- There is a bunch of checkstyle errors. For an initial release it would
> be good to start with a clean code base.

With the exception of one method with errors I accidentally introduced just
before the release, these were all either test code or red herrings. For
this [weaver] will no longer publish an aggregate checkstyle report, but
the modules for which checkstyle is relevant will publish their own
specific reports.

> - From the documentation on the site it is hard to get an understanding
> what this component does and how it can be used in practice. Maybe a
> short "getting started guide" or "some steps to start your project"
> would be helpful (I know this is much work to do and less fun than coding).

Some work has begun in this regard on the site.

I've never seen the AL header in release notes before.
> Not sure that's necessary (and it makes the notes harder to read).
> This has been fixed.

> The poms don't include any groupId definitions.

I do not plan to address this as it seems to be common practice in the
Maven-using world to omit a redundant groupId.

I would also like to see reports for FindBugs and PMD. Also: use
> commons-parent 33 to pick up the new skin.

These items are done.

Please review the release candidate and vote.
  This vote will close no sooner that 72 hours from now, i.e. after 0100UTC
8-March 2014

  [ ] +1 Release these artifacts
  [ ] +0 OK, but...
  [ ] -0 OK, but really should fix...
  [ ] -1 I oppose this release because...


  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message