Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-09-05 Thread Jakob Homan
+1 (binding). Sorry for the late vote; was on holiday. This release has gotten a bit complicated in that it didn't actually end up with 3 binding votes. Going to add mine now. * MD5/SHAs check out. * LICENSE/NOTICE/DISCLAIMER look good * spot check on licenses looks good. * setup works. -Jakob

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-07 Thread Bolke de Bruin
Also the author replied and confirmed a BSD license: https://github.com/novus/nvd3/issues/2064#issuecomment-313718380 Kind regards Bolke Sent from my iPhone > On 7 Jul 2017, at 03:17, Justin Mclean wrote: > > Hi, > >> The patch that contains “bullet.js” was committed on Wed May 25 >> (https

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Justin Mclean
Hi, > The patch that contains “bullet.js” was committed on Wed May 25 > (https://github.com/novus/nvd3/commit/b7670eeaffe45a23bae1797306750c2b6ae8bcce > > ), > thus after the removal of the GPL-license file, and spe

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread John D. Ament
And to answer the question why this only came up now. I'm mentoring the Aria Tosca project, and looking at their release pointed out some of these issues, so only then I came to find out the differences about this. Thanks to Ran, I understand better now. John On Thu, Jul 6, 2017 at 1:38 PM Bolk

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Bolke de Bruin
Thanks alot. That does indeed help, and might be a structure we will start using ourselves. Cheers Bolke Sent from my iPhone > On 6 Jul 2017, at 17:34, Ran Ziv wrote: > > Hi, > > I'm not 100% sure this is relevant, but regarding sdist vs source tarball, > we just had a similar issue in the

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Ran Ziv
Hi, I'm not 100% sure this is relevant, but regarding sdist vs source tarball, we just had a similar issue in the apache-ariatosca project when the release candidate package I had brought up to a vote contained a Pythonic sdist rather than a source code tarball. We ended up creating three differen

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Bolke de Bruin
Protovis became d3, which is BSD licensed. D3’s examples contain the same code. The author (Mike Bostock) is the same in both cases. I have just send him an email, I hope he responds. Cheers Bolke > On 6 Jul 2017, at 15:08, Justin Mclean wrote: > > HI, > >> [20] “nv.d3.js” originates from h

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Bolke de Bruin
Hi Justin, That’s for the whole project nvd3, which was then also dual licensed APL2. Next to that this file has existed only for two months in this repository: Bolkes-MacBook-Pro:nvd3 bolke$ git log --all --full-history -- GPL-LICENSE.v3.txt commit 8e52a5743e843015495da495f83153ac5552edf3 Auth

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Justin Mclean
Hi, > I have looked at the history of nvd3 “bullet” implementation and its history > says it is copied from d3-examples > (https://github.com/novus/nvd3/commit/b7670eeaffe45a23bae1797306750c2b6ae8bcce > > ) > whic

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Justin Mclean
HI, > [20] “nv.d3.js” originates from http://nvd3.org and its > license states it is Apache License 2, > https://github.com/novus/nvd3/blob/master/LICENSE.md > . And as far as I > understand it it should not need to be l

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Bolke de Bruin
I have looked at the history of nvd3 “bullet” implementation and its history says it is copied from d3-examples (https://github.com/novus/nvd3/commit/b7670eeaffe45a23bae1797306750c2b6ae8bcce ) which is located at:

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Bolke de Bruin
Hi Justin, [20] “nv.d3.js” originates from http://nvd3.org and its license states it is Apache License 2, https://github.com/novus/nvd3/blob/master/LICENSE.md . And as far as I understand it it should not need to be lice

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-06 Thread Justin Mclean
Hi, > The situation isn't that bad. It is due to the fact this is a "sdist" (not a > bdist). A single -1 vote isn’t bad and other IPMC member may vote +1 - but probably unlikely in this case I think. Whatever the artefact is called the LICENSE and NOTICE needs to reflect what is contained wit

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-05 Thread Bolke de Bruin
The situation isn't that bad. It is due to the fact this is a "sdist" (not a bdist). Our git repo does contain the licenses and does pass RAT. I'll discuss with Max how we will fix this. Probably we create a real "source tarball" with build instructions to create a "sdist". Let's see. I'm just

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-05 Thread Justin Mclean
Hi, Sorry but it’s -1 form me due to licensing issues. If it was just LICENSE missing a few permissive licensed pieces I would be +1 please fix in next release. However you are not following the terms of the MIT license (missing the MIT text in several cases) and their’s one file which may incl

Re: [VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-03 Thread John D. Ament
Hi, I have a few questions about the release contents. I ran RAT, and found many unapproved files. Are you maybe missing a few things from your rat excludes? https://paste.apache.org/jBql is the report I got (ignore rat-output, it's the file I was piping to). Note that there is no rat-excludes

[VOTE] Release Apache Airflow 1.8.2 (incubating)

2017-07-03 Thread Maxime Beauchemin
Hello Incubator PMC’ers, The Apache Airflow community has voted and approved the proposal to release Apache Airflow 1.8.2 (incubating) based on 1.8.2 Release Candidate 2. We now kindly request the Incubator PMC members to review and vote on this incubator release. If the vote is successful we will