commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Gregory <>
Subject Re: [VOTE] Release Logging 1.2 based on RC1
Date Sat, 28 Jun 2014 16:45:10 GMT
Yes, I agree about not fixing all issues found by the new reports. The reports are more for
our own need going forward. Perhaps we can make a note of the fact that issues found by the
reports are known and are just tips for current developers and anyone else wanting to get
their feet wet in the project with some small changes.


<div>-------- Original message --------</div><div>From: Thomas Neidhart
<> </div><div>Date:06/28/2014  10:40  (GMT-05:00)
</div><div>To: Commons Developers List <> </div><div>Subject:
Re: [VOTE] Release Logging 1.2 based on RC1 </div><div>
</div>On 06/28/2014 03:27 PM, Gary Gregory wrote:
> On Sat, Jun 28, 2014 at 9:25 AM, Gary Gregory <>
> wrote:
>> On Sat, Jun 28, 2014 at 9:23 AM, Gary Gregory <>
>> wrote:
>>> On Sat, Jun 28, 2014 at 9:01 AM, Thomas Neidhart <
>>>> wrote:
>>>> On 06/28/2014 02:45 PM, Gary Gregory wrote:
>>>>> So -1 because the pmd.xml file is missing from the src zip.
>>>> r1484126 | ggregory | 2013-05-18 17:15:22 +0200 (Sat, 18 May 2013) | 1
>>>> line
> and wow, we have not cut a release in a long time or checked the build from
> the src zip!

the last release was about 1 1/4 years ago. Since then the pmd and
findbugs reports have been added without properly checking that the
release build still is complete. Ok can happen.

>>>> Add FindBugs and PMD reports.
>>>> Ha, you added the reports, and I did not notice it. That is unfortunate.
>>>> btw. I did disable the findbugs report again as it was mainly creating
>>>> false positives and I did not want to spend the time adding exclude
>>>> filters.
>>> Hm... I like having the reports as ideas for fixing things if I am idle
>>> or to give others ideas on how to get their feet wet with small changes.

sure, if time permits it, but it should not block the release for a
component that is unchanged for years.

>> I just added the pmd.xml to the src assembly in trunk.

ok thanks.


To unsubscribe, e-mail:
For additional commands, e-mail:

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