incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexander Bezzubov <abezzu...@nflabs.com>
Subject Re: [VOTE] Release Apache Zeppelin (incubating) 0.5.0-incubating
Date Wed, 22 Jul 2015 01:41:05 GMT
Apologies for not timely response.

Thank you Konstantin and Justin for your reviews - we have filed the
issues to track that your feedback is going to be addressed
https://issues.apache.org/jira/browse/ZEPPELIN-174
https://issues.apache.org/jira/browse/ZEPPELIN-175

Regarding the issue raised by Edward - we were under impression that
this particular contribution is covered by
https://github.com/apache/incubator-zeppelin/blob/master/NOTICE#L53
(as we use the code from
https://code.google.com/p/selenium/issues/detail?id=1361 that was not
merged to the selenium, keeping the attribution)

Please let me know in case if that is not enough and we will be glad
to address it asap to reconcile the -1.

Thanks in advance!

On Wed, Jul 22, 2015 at 2:46 AM, Henry Saputra <henry.saputra@gmail.com> wrote:
> Thanks to Justin for his thorough review, as always.
>
> I have pinged back the Apache Zeppelin community to see if we could
> quickly address the issue.
>
> - Henry
>
> On Mon, Jul 20, 2015 at 9:58 PM, Edward J. Yoon <edwardyoon@apache.org> wrote:
>>> Has the author of zeppelin-server/src/test/java/com/webautomation/ScreenCaptureHtmlUnitDriver.java
signed a ICLA?
>>
>> It looks like came from Selenium community -
>> https://groups.google.com/forum/#!searchin/selenium-developers/Ivan$20Vasiliev$20%7Csort:relevance/selenium-developers/PTR_j4xLVRM/k2yVq01Fa7oJ
>>
>> If license header can't be attached discretionally by a third party, I
>> don't think this can be included. -1 (binding) I think it'd be nice to
>> fix every LICENSE and NOTICE issues.
>>
>>
>> On Sun, Jul 19, 2015 at 9:33 AM, Justin Mclean <justin@classsoftware.com> wrote:
>>> Hi,
>>>
>>> +1 binding
>>>
>>> Please fix the LICENSE/NOTICE issues are fixed for next release. Given everything
involved is Apache comparable it's more a documentation issue but still important to get right.
If you need a hand or need it reviewed before your next release just ask.
>>>
>>> The LICENSE/NOTICE issues and missing DISCLAIMER from the binary files is more
serious and IMO they shouldn’t be released but I’ll leave that up to the PMC/IPMC.
>>>
>>> I checked:
>>> - signatures and hashes good
>>> - incubating in source file
>>> - DISCLAIMER exists
>>> - LICENSE isn’t correct (see below)
>>> - NOTICE also has issues (see below)
>>> - All source has apache headers (except one)
>>> - No unexpected binary files
>>> - Can compile from source
>>>
>>> LICENSE and NOTICE issues
>>> - LICENSE states "Copyright 2014, NFLabs inc.” when it should be ASF and specified
in the NOTICE file.
>>> - NOTICE contains Font Awesome (SIL + MIT), Ace (BSD), Simple line icons (MIT)
but there is no need for these to be in NOTICE [1]
>>> - LICENSE should contain things mentioned in NOTICE [1]
>>> - Has the author of  zeppelin-server/src/test/java/com/webautomation/ScreenCaptureHtmlUnitDriver.java
signed a ICLA? I notice it’s missing a header and it’s not 100% clear to me what it’s
licensing is.
>>>
>>> For the binary release:
>>> - Missing NOTICE and DISCLAIMER from top level directory
>>> - Unadorned LICENSE and NOTICE in META_INF or jar and war so not correct, please
see [2]. There's several NOTICE files in the libs jars that would have an impact on the binary
NOTICE file [3]
>>>
>>> Thanks,
>>> Justin
>>>
>>> 1. http://www.apache.org/dev/licensing-howto.html#permissive-deps
>>> 2. http://www.apache.org/dev/licensing-howto.html#binary
>>> 3. http://www.apache.org/dev/licensing-howto.html#alv2-dep
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>
>>
>>
>>
>> --
>> Best Regards, Edward J. Yoon
>>
>> ---------------------------------------------------------------------
>> 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
>



-- 
--
Kind regards,
Alexander.

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


Mime
View raw message