We miss one vote for this vote to conclude. Any IPMC member available to vote 
on this?

> On Jul 21, 2019, at 09:13, Antoine Toulme <anto...@toulme.name> wrote:
> 
> Acknowledged. We will fix those issues in the next release.
> 
>> On Jul 19, 2019, at 21:02, Justin Mclean <jus...@classsoftware.com> wrote:
>> 
>> Hi,
>> 
>> +0 (binding)
>> 
>> This release includes Category X software (which is incompatible with ALv2) 
>> and it's not following the terms of the GPL license. While this thread [12] 
>> indicates incubating releases can include Category X they still need to 
>> follow the terms of the included licenses.  This is also dependent on the 
>> incubator being “special” and it releases allowed to not confirm to policy 
>> (unlike other TLP projects) this is this still as yet to be 
>> confirmed/allowed.
>> 
>> I checked:
>> - incubating in name
>> - disclaimer exists
>> - LICENSE has a few minor issues
>> - NOTICE is fine
>> - Includes Category X licensed software [4] It also looks like these files 
>> [6][7][8][9] come from [10][11] and are also GPL licensed.
> These are coming from the ethereum tests repo. I think we will find a new way 
> to package sources without those tests and check out the git repo at test 
> time then. Does that work?
>> - No unexpected binary files
>> - A few source files (.sh and piton scripts) are missing headers (how I came 
>> across [4]), would be good to add them.
>> 
>> In LICENSE I noticed that:
>> - This file [1] is under ALv2 but they have modified the license [2] and 
>> added an extra condition (top of file) so may not be ALv2 compatible. Please 
>> check with legal discuss.
> Both web3j and Tuweni took this class from the Bitcoinj project. The file 
> mentions its origins. I can change the header as needed.
>> - There files seem to have incorrect headers [1][3] as they have not been 
>> licensed to the ASF. I believe they should have the AL 3rd party header.
> Got it, will fix, thanks!
>> - The license mentions the ProgPoW algorithm but it unclear where it’s being 
>> used. I assume here [5] but there's nothing in that file that indicate it 
>> contains 3rd party code or that it’s CC0 licensed.
> OK we will fix.
>> 
>> Thanks,
>> Justin
>> 
>> 1. crypto/src/main/java/org/apache/tuweni/crypto/SECP256K1.java
>> 2. https://github.com/web3j/web3j/blob/master/LICENSE
>> 3. crypto/src/main/java/org/apache/tuweni/crypto/SECP256K1.java
>> 4. eth-reference-tests/src/test/resources/tests/ansible/ec2.py
>> 5. ./progpow/src/main/java/org/apache/tuweni/progpow/ProgPoW.java
>> 6. 
>> /eth-reference-tests/src/test/resources/tests/ansible/test-files/docker-cpp/Docker-file
>> 7. 
>> /eth-reference-tests/src/test/resources/tests/ansible/test-files/docker-cppjit/Dockerfile
>>     
>> 8  
>> ./eth-reference-tests/src/test/resources/tests/ansible/test-files/docker-python/Dockerfile
>> 9 
>> ./eth-reference-tests/src/test/resources/tests/ansible/test-files/docker-go/Dockerfile
>> 10. https://github.com/ethereum/aleth/blob/master/LICENSE
>> 11. https://github.com/ethereum/go-ethereum/blob/master/COPYING
>> 12. 
>> https://lists.apache.org/thread.html/68e65125bcab2ac812f1359ccfc79acdf94f244416fdda52328b256e@%3Clegal-discuss.apache.org%3E
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
>> For additional commands, e-mail: general-h...@incubator.apache.org
>> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
> 


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

Reply via email to