Hi Justin

The root NOTICE references the donation but files are under apache v2
license anyway, even referencing IBM so should be good.

Romain

Le 11 déc. 2017 07:02, "Justin Mclean" <jus...@classsoftware.com> a écrit :

Hi,

I’m not 100% sure I would vote +1 on this release due to possible header
issues (copyright IBM) and missing license information, but it’s probably
in the “please fix for next release” category. Where these files with IBM
headers a part of the donation or do they come from other 3rd party code?
If so does that other 3rd party code have NOTICE files?

What do other IPMC members think?

I checked:
- incubating in name
- hashes and signatures correct
- disclaimer exists
- LICENSE is missing a couple of things (see below)
- NOTICE looks good
- A number of files (120 or so ) are still "Copyright 2012 International
Business Machines Corp.”  and don’t have ASF headers. I assume these files
were part of the donation from IBM and just been missed? Two examples [3][4]
- no binary files in release
- can compile from source

LICENSE is missing a couple of permissive licenses.
- bootstrap [1][2] . While this is the older Apache license of bootstrap
(so it's not required to be added to LICENSE), more recent versions are MIT
licensed so might be good to call that out in the LICENSE. Bootstrap also
contains MIT licensed normalize.css.
- MIT licensed JQuery [5] which also contains MIT licensed Sizzle.js

And a very minor thing I notice some dependancies files refer to ALv2 as
the “Apache Public License 2.0” - not GPL thankfully :-) e.g. [6] but I
assume this is auto generated.

Thanks,
Justin

1. batchee-0.5-incubating/gui/servlet/embedded/src/main/
resources/META-INF/resources/internal/batchee/css/bootstrap.min.3.0.0.css
2. batchee-0.5-incubating/gui/servlet/embedded/src/main/
resources/META-INF/resources/internal/batchee/js/bootstrap.min.3.0.0.js
3. batchee-0.5-incubating/jbatch/src/main/xsd/jobXML_1_0.xsd
4. batchee-0.5-incubating/jbatch/src/main/java/org/apache/
batchee/container/impl/controller/chunk/RetryHandler.java
5. batchee-0.5-incubating/gui/servlet/embedded/src/main/
resources/META-INF/resources/internal/batchee/js/jquery-2.0.3.min.js
6. ./tools/maven-plugin/target/classes/META-INF/DEPENDENCIES


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

Reply via email to