-1

There are a few issues:
1) The features.xml still includes the geronimo-osgi-registry bundle which 
isn't needed anymore.   This is minor and I'm definitely OK with leaving it 
there.   This wouldn't warrant a -1.

2) The sts example has a test failure on Java 5 (It's OK on Java6).   Colm is 
looking into it, but again, not something I would -1 and I'd be OK with not 
having that fixed immediately.

HOWEVER:

3) The release was done with Java 6.  I've never done that before (I've always 
used Java5) and thus this was not really tested.  The GOOD news is that all 
the jars are OK.  However, the lib directory in the distribution is missing 
some stuff:
 geronimo-activation_1.1_spec-1.1.jar
 geronimo-annotation_1.0_spec-1.1.1.jar
 geronimo-stax-api_1.0_spec-1.0.1.jar
 geronimo-ws-metadata_2.0_spec-1.1.3.jar
 jaxb-api-2.2.3.jar
 mimepull-1.4.jar
 saaj-api-1.3.4.jar
 saaj-impl-1.3.12.jar

Thus, pretty much nothing in the distribution works on Java5.   You cannot 
even run "wsdl2java --version" or anything as you get class not founds and 
such.    The good news is this wouldn't affect the "release:prepare" step.  
You don't have to re-do all of that.   You can just drop this repository, 
cancel this vote, checkout the tag and do "mvn deploy -
Pdeploy,release,everything,jaxws22 -Dmaven.test.skip.exec=true" using Java5 to 
restage things.   A little simpler and quicker than a full prepare/perform 
cycle.


Dan



On Friday, November 04, 2011 12:00:35 AM Freeman Fang wrote:
> Hi All,
> 
> We've resolved over 35 issues since 2.4.3 and thus is time for a
> release.
> List of issues:
> 2.4.4:
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310511&ve
> rsion=12318347
> 
> The Maven staging areas are at:
> 2.4.4:
> https://repository.apache.org/content/repositories/orgapachecxf-152/
> 
> 
> The distributions are in the org/apache/cxf/apache-cxf/ directory of
> the Maven
> staging areas.
> 
> This release is tagged at:
> http://svn.apache.org/repos/asf/cxf/tags/cxf-2.4.4
> 
> 
> Please vote to approve this release:
> 
> [ ] +1 Approve the release
> [ ] -1 Veto the release (please provide specific comments)
> 
> This vote will be open for at least 72 hours.
> 
> Best Regards
> Freeman
> 
> ---------------------------------------------
> Freeman Fang
> 
> FuseSource
> Email:[email protected]
> Web: fusesource.com
> Twitter: freemanfang
> Blog: http://freemanfang.blogspot.com
-- 
Daniel Kulp
[email protected]
http://dankulp.com/blog
Talend - http://www.talend.com

Reply via email to