Thank you for a thorough review!

We've started tracking the Category B issues at
https://issues.apache.org/jira/browse/TAVERNA-927
We would appreciate your insight and help as we are a bit short on
active mentors! :-)



The ZIP files under the test directory needs to be zipped because the
Taverna Language deals (in part) with reading and writing a
zip-derived file format.

I've fixed in git master the workflowrun.bundle.zip internal LICENSE
file which was missing the appendix so it should be fine for next
release.


As for code signature, the release candidate and Maven artifacts are
signed by st...@apache.org A0FFD119 which is listed in KEYS. I
switched my primary UID so that st...@apache.org is on the top and
updated
https://dist.apache.org/repos/dist/release/incubator/taverna/KEYS
(See http://unix.stackexchange.com/a/153310 )


(BTW - we used to update KEYS from
https://people.apache.org/keys/group/taverna.asc to make sure that
id.apache.org had the openpgp fingerprint - but this has gone 404 in
the recent update of home.apache.org together with most incubator
groups)

On 5 March 2016 at 23:02, Justin Mclean <jus...@classsoftware.com> wrote:
> Hi,
>
> Sorry it -1 binding due to Category B software in a source release. [10]
>
> I checked:
> - artefacts contain incubating
> - signature and hashes good
> - DISCLAIMER exists
> - LICENSE is good however it lists Category B software
> - NOTICE is good
> - There are a couple of binary files (under the test directory) do these need 
> to be zipped?
> - All source files have headers
>
>
> There are some licensing issues:
> - Several items [3][4][5][6] are licensed under W3C document license, this is 
> not a category A license and looks like they can only be included in binary 
> release [2]. (i.e. Category B but it's not listed) You may need to clarify 
> this on legal discuss.
> - Files [8][9][16][17][18][19][20] licensed under CC-BY-3 can only be 
> included in a binary release. [10]
> - I assume CC-A [11][12] would also need to be handled the same way.
> - It is unclear how [13] is licensed.
> - Looks like [14][15] are not licensed under a Category A license. Again 
> probably needs to be discussed on legal discuss.
>
> The LICENSE and NOTICE in ./taverna-tavlang-tool/src/main/resources/META-INF/ 
> may need updating, looks like the contents don’t match the top level license 
> and year is incorrect.
>
> Also:
> - Please consider signed the artefacts with an apache email address.
> - The workflowrun.bundle.zip [1] file contains a LICENSE which is missing the 
> appendix
>
> Thanks,
> Justin
>
> 1. 
> apache-taverna-language-0.15.1-incubating/taverna-robundle/src/test/resources/workflowrun.bundle.zip
> 2. https://issues.apache.org/jira/browse/LEGAL-111
> 3. 
> ./taverna-scufl2-schemas/src/main/resources/org/apache/taverna/scufl2/rdfxml/xsd/xml.xsd
> 4. ./taverna-robundle/src/main/resources/ontologies/prov-aq.rdf
> 5. ./taverna-robundle/src/main/resources/ontologies/prov-o.rdf
> 6. ./taverna-robundle/src/main/resources/ontologies/prov-o.rdf
> 7. ./taverna-robundle/src/main/resources/ontologies/dcam.owl
> 8. ./taverna-robundle/src/main/resources/ontologies/dcam.owl
> 9. ./taverna-robundle/src/main/resources/ontologies/dcterms_od.owl
> 10. http://www.apache.org/legal/resolved.html#category-b
> 11. ./taverna-robundle/src/main/resources/ontologies/foaf.rdf
> 12. ./taverna-scufl2-wfdesc/src/main/resources/com/xmlns/foaf/foaf.rdf
> 13. ./taverna-robundle/src/main/xsd/xenc-schema.xsd
> 14. ./taverna-robundle/src/main/resources/ontologies/oa.rdf
> 15. ./taverna-robundle/src/main/resources/ontologies/ro.owl
> 16. 
> ./taverna-scufl2-wfdesc/src/main/resources/org/purl/wf4ever/wfdesc/wfdesc.ttl
> 17. 
> ./taverna-scufl2-wfdesc/src/main/resources/org/purl/wf4ever/wfdesc/wfprov.ttl
> 18. 
> ./taverna-scufl2-wfdesc/src/main/resources/org/purl/wf4ever/wfdesc/roterms.ttl
> 19. 
> ./taverna-scufl2-wfdesc/src/main/resources/org/purl/wf4ever/wfdesc/wf4ever.ttl
> 20 
> ./taverna-scufl2-schemas/src/main/resources/org/apache/taverna/scufl2/rdfxml/xsd/roevo.xsd
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>



-- 
Stian Soiland-Reyes
Apache Taverna (incubating), Apache Commons RDF (incubating)
http://orcid.org/0000-0001-9842-9718

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

Reply via email to