incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: [VOTE] Release Apache Groovy 2.4.4-incubating
Date Tue, 14 Jul 2015 19:46:33 GMT
+1 (binding)

> On Jul 13, 2015, at 10: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