incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Roman Shaposhnik <ro...@shaposhnik.org>
Subject Re: [VOTE] Release Apache Groovy 2.4.4-incubating
Date Mon, 13 Jul 2015 16:54:15 GMT
Hi!

first all, I'd like to say that between the last RC that
I reviewed and this one Groovy team has made
huge progress. You guys rock! That said, the IP
hygiene is one of the biggest parts of the curriculum
known as Incubator. For that reason, I can't thank
Justin enough for his thorough review.

As it appears to me his feedback shouldn't be that
difficult to take care of. Once again, I do realize that
RCs keep coming which could be a bit frustrating,
but hopefully the next one will be the last one.

As such, I'm -1 on this RC. Blockers for my vote are
all around L&N files. The rest of the feedback, while
super useful, will not be critical for me to change my
vote back to +1.

-1 (binding).

Thanks,
Roman.

On Mon, Jul 13, 2015 at 7:23 AM, Emmanuel Lécharny <elecharny@gmail.com> wrote:
> Hi,
>
> my +1 (Binding), for the source package.
>
> Addressing Justin's issues :
>
> There *are* issues, but considering the security issue fixed in this
> release, I'd rather have this version out.
>
> - The build scripts are failing because there are Windows file (^M at
> the end of ech line). Removing them let you build the project. This has
> to be fixed, though.
> - NOTICE : not critical, IMO, but need to be fixed.
> - LICENSE : I can't find the normalize.css file. The MIT & BSD license
> should be added into LICENSE. The not bundled licenses should be removed.
>
> All in all, there are issues, that need to be addressed, and I expect
> them to be fixed in the next release.
>
> Regarding the binary package, it has to contain the N&L files. I have
> not checked it, because they are by-product, but that was a mistake
> (obviously, people will use them instead of using the sources). I would
> -1 the binary package as of today.
>
> Thanks !
>
> Le 13/07/15 14:12, Justin Mclean a écrit :
>> Hi,
>>
>> -1 (binding) as LICENSE and NOTICE have issues, included files which have Apache
header when they are licensed under other terms, and binary connivence files are missing required
files (i.e. DISCLAIMER, LICENSE and NOTICE) Note that the binary LICENSE and NOTICE file are
very likely different to the the source LICENSE and NOTICE files.
>>
>> For the source release I checked:
>> - signatures ok but should be signed by apache.org address
>> - hashes good
>> - DISCLAIMER exists
>> - LICENSE and NOTICE have issues (see below)
>> - No unexpected binaries in source release
>> - All source files have Apache header
>> - Probably my setup/config but unable to compile from source and get this error:
>>
>> FAILURE: Build failed with an exception.
>> * Where:
>> Script '/Users/justinmclean/Downloads/ApacheGroovy/groovy-2.4.4/gradle/asciidoctor.gradle'
line: 19
>> * What went wrong:
>> A problem occurred evaluating script.
>>> Could not create task of type 'AsciidoctorTask'.
>>  LICENSE and NOTICE issues:
>> - NOTICE contains items that are not required (as they are not bundled) and it’s
not in usual format
>> - LICENSE is missing:
>>       - MIT licensed asciidoctor.org see /src/spec/assets/css/style.css
>>       - MIT licensed normalize.css (in two places)
>>       - BSD licensed FileNameCompleter.groovy which also has an Apache header
>> - LICENSE also contains several licenses/items that are not bundled and so shouldn’t
be included e.g. ANTLR 2, ASM 4, Hamcrest, JLine, JSR223, JUnit, Multiverse
>>
>> For binary releases:
>> - All are missing DISCLAIMER, LICENSE and NOTICE
>>
>> Other issues:
>> - release candidate not in correct place
>> - not signed by apache.org address
>> - Short form of bundled licenses are preferred to long version
>> - all zips unzip to same directory
>>
>> Thanks,
>> Justin
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: general-help@incubator.apache.org
>>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Mime
View raw message