flink-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stephan Ewen <se...@apache.org>
Subject Extended License entries.
Date Mon, 22 Sep 2014 20:57:49 GMT
Hi Justin!

I addressed your comments.

We actually have two pairs LICENSE and NOTICE files, one for the source
(containing only bundled source code, mainly CSS, JS, Fonts) and one pair
for the binary release (with all bundled bits).

I added versions and licence notices for transitive sources (normalize.css,
sizzle.js). Not sure if strictly required, but better safe than sorry.


On Mon, Sep 22, 2014 at 5:58 AM, Justin Mclean <justin@classsoftware.com>

> Hi,
> +1 binding
> Here's what I checked
> - Vote good
> - artefact has incubating in name
> - MD5 and signatures correct
> - NOTICE and LICENSE good with a couple of minor issues (see below)
> - All source files have correct headers
> - No unexpected binary files in source package
> - can compile from source package
> Some minor things perhaps you can look at fixing in next release.
> - "depends on and/or bundles" make it hard to review and what you need to
> add to license depends on it it bundled or not. Any chance you could just
> list the MIT/BSD bundled bits in LICENSE. I assume this was done so that
> you didn't need separate LICENSE/NOTICE files for the source and binary
> releases?
> - Specify version of bootstrap used. From a quick glance look like you
> have multiple versions ie 3.1.0 and 3.1.1. As bootstrap changed license
> around this time it would useful to list the version number.
> - Looks like you may have multiple versions of jQuery as well eg
> (flink-0.6.1-incubating/flink-runtime/resources/web-docs-infoserver/js/jquery-1.10.2.js
> and
> flink-0.6.1-incubating/flink-clients/resources/web-docs/js/jquery-2.1.0.js).
> These versions are not mentioned in LICENSE.
> - May be missing a licence for normalise.css (see comment in
> /flink-0.6.1-incubating/flink-clients/resources/web-docs/css/bootstrap.css).
> Not sure what's required here.
> - May be misisng a licence for sizzle.js (see comment at head of
> flink-0.6.1-incubating/flink-runtime/resources/web-docs-infoserver/js/jquery.js
> ). Not sure what's required here.
> - May not need
> flink-0.6.1-incubating/flink-clients/resources/web-docs/img/StratosphereLogo.png?
> Or should that be changed to be a Flink logo?
> Thanks,
> Justin
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org

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