commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Simone Tripodi <simonetrip...@apache.org>
Subject [RESULT][VOTE] Release Apache Commons-Digester 3.1 based on RC2
Date Fri, 28 Oct 2011 21:50:52 GMT
Hi all guys,
Since 72 hours passed I'm declaring this vote as closed and passed
with 3 positive votes from PMCs:

 * Simone Tripodi
 * Luc Maisonobe
 * Oliver Heger

no other votes were cast.

I'll move the proposed release to dist/ and give 24 hours for mirrors
to sync before making ANN and site update.

Thanks to all for your support and patience!
Have a nice weekend, all the best!
Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/



On Thu, Oct 27, 2011 at 9:00 AM, Simone Tripodi
<simonetripodi@apache.org> wrote:
> Hi Gary!!
> I tried to force the PMD plugin to version 2.5 in the Digester
> pom.xml, unfortunately looking at the effective pom I realized that
> the site-plugin-3.0 continues using the pmd-plugin-2.3, because it is
> configured inside the reportset :(
> I'll have a deep look at the site-plugin.
> Thanks for the hint!
> Simo
>
> http://people.apache.org/~simonetripodi/
> http://simonetripodi.livejournal.com/
> http://twitter.com/simonetripodi
> http://www.99soft.org/
>
>
>
> On Thu, Oct 27, 2011 at 12:54 AM, Gary Gregory <garydgregory@gmail.com> wrote:
>> Simo,
>>
>> I'm on my phone ATM so I cannot check this, but are you using the
>> latest versions of the plugins? Maybe some of these false positives
>> would go away.
>>
>> Gary
>>
>> On Oct 25, 2011, at 17:39, Simone Tripodi <simonetripodi@apache.org> wrote:
>>
>>> Hi Gary!
>>>
>>> That is the same I wondered: if 6 are "too many", how many are enough? :)
>>> Maybe it's time PMD updates its rules? :P
>>>
>>> About the other PMD violations:
>>>
>>> `Avoid unused private methods such as 'npeSafeCast(Object)'` is not
>>> true, it is used:
>>>
>>> 2624     public <T> T peek()
>>> 2625     {
>>> 2626         try
>>> 2627         {
>>> 2628             return this.<T> npeSafeCast( stack.peek() );
>>> 2629         }
>>> 2630         catch ( EmptyStackException e )
>>> 2631         {
>>> 2632             log.warn( "Empty stack (returning null)" );
>>> 2633             return ( null );
>>> 2634         }
>>> 2635     }
>>>
>>> `Overriding method merely calls super` is not true, it returns a
>>> method chain that involves the super:
>>>
>>> 49      @Override
>>> 50      protected LinkedRuleBuilder forPattern( String pattern )
>>> 51      {
>>> 52          return super.forPattern( pattern ).withNamespaceURI( namespaceURI
);
>>> 53      }
>>>
>>> `Avoid empty catch blocks` is the same that we have in IO:
>>>
>>> 77              try
>>> 78              {
>>> 79                  if ( s != null )
>>> 80                  {
>>> 81                      s.close();
>>> 82                  }
>>> 83              }
>>> 84              catch ( IOException e )
>>> 85              {
>>> 86                  // close quietly
>>> 87              }
>>>
>>> Thanks for reviewing, hope this clarifies and you will express a +1 :P
>>> All the best,
>>> Simo
>>>
>>> http://people.apache.org/~simonetripodi/
>>> http://simonetripodi.livejournal.com/
>>> http://twitter.com/simonetripodi
>>> http://www.99soft.org/
>>>
>>>
>>>
>>> On Tue, Oct 25, 2011 at 11:24 PM, Gary Gregory <garydgregory@gmail.com>
wrote:
>>>> Hi Simo,
>>>>
>>>> Not blockers but the PMD issues could be addressed except:
>>>>
>>>> Should we turn off "Too many static imports may lead to messy code"? How
>>>> many is too many?
>>>>
>>>> Gary
>>>>
>>>> On Tue, Oct 25, 2011 at 4:10 PM, Simone Tripodi <simonetripodi@apache.org>wrote:
>>>>
>>>>> Hi all guys,
>>>>> I'm writing to call for a vote to release apache commons-digester-3.1
>>>>> based on RC2.
>>>>>
>>>>> This is the 3.1RC1 release with fixed doubts were raise in the first
VOTE
>>>>> call.
>>>>> Please take in consideration that broken 3.1 links will be fixed once
>>>>> the site will be deployed.
>>>>>
>>>>> The vote will stay open for 72 hours and closes on Friday 28th, at 8:10pm
>>>>> CET.
>>>>>
>>>>> Many thanks in advance for reviewing, have a nice day!
>>>>> All the best,
>>>>> Simo
>>>>>
>>>>> Release notes:
>>>>>
>>>>>
>>>>> http://people.apache.org/builds/commons/digester/3.1/RC2/RELEASE-NOTES.txt
>>>>>
>>>>> Tag:
>>>>>
>>>>>
>>>>> https://svn.apache.org/repos/asf/commons/proper/digester/tags/DIGESTER3_3_1_RC2/
>>>>>
>>>>> Site:
>>>>>
>>>>>  http://people.apache.org/builds/commons/digester/3.1/RC2/site/
>>>>>
>>>>> Binaries:
>>>>>
>>>>>  http://people.apache.org/builds/commons/digester/3.1/RC2/binaries/
>>>>>
>>>>> Maven Artifacts (staged on Nexus)
>>>>>
>>>>>
>>>>> https://repository.apache.org/content/repositories/orgapachecommons-098/org/apache/commons/commons-digester3/
>>>>>
>>>>> [ ] +1 release it
>>>>> [ ] +0 go ahead I don't care
>>>>> [ ] -1 no, do not release it because... (please explain why)
>>>>>
>>>>> http://people.apache.org/~simonetripodi/
>>>>> http://simonetripodi.livejournal.com/
>>>>> http://twitter.com/simonetripodi
>>>>> http://www.99soft.org/
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>>>> For additional commands, e-mail: dev-help@commons.apache.org
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> E-Mail: garydgregory@gmail.com | ggregory@apache.org
>>>> JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
>>>> Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
>>>> Blog: http://garygregory.wordpress.com
>>>> Home: http://garygregory.com/
>>>> Tweet! http://twitter.com/GaryGregory
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>> For additional commands, e-mail: dev-help@commons.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> For additional commands, e-mail: dev-help@commons.apache.org
>>
>>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message