You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by Rodric Rabbah <ra...@apache.org> on 2019/06/09 12:53:40 UTC
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
For several of the issues you noted, I opened github defects [1-4] against the relevant repos so that we will address them before the next release of the corresponding artifacts. We've also discussed the use of "short licenses" [5] and we document our use of the short licenses and license exclusions (noted for gradle) here [6] per Apache guidance [7].
[1] https://github.com/apache/incubator-openwhisk-runtime-docker/issues/69
[2] https://github.com/apache/incubator-openwhisk-runtime-python/issues/58
[3] https://github.com/apache/incubator-openwhisk-runtime-go/issues/87
[4] https://github.com/apache/incubator-openwhisk-runtime-go/issues/91
[5] https://lists.apache.org/thread.html/bbec59cc560d446a4239627dcb463bc95fb98be5eb66d525f93e5ef1@%3Cdev.openwhisk.apache.org%3E
[6] https://github.com/apache/incubator-openwhisk-release/blob/master/docs/license_compliance.md
[7] https://www.apache.org/legal/src-headers.html
> - The go LICENSE incorrectly states you should add "Copyright 2015-2016 IBM Corporation” to your own files
> - the text of the license for Filetype 1.0.5 is not included (as required by it's license). The pointer to the licenses should point to a copy of the license in distribution as licenses and URLs can change over time.
The license was corrected and is now the Apache license with no added text. The references to filetype were no longer relevant but
> - How is this file licensed? [3] Even if is it a “free” license it may have terms around distribution that make it incompatible with the Apache license, and even if not then it still good to list it in LICENSE.
>
> Thanks,.
> Justin
>
> 1. ./incubator-openwhisk-runtime-docker-1.13.0-incubating/sdk/docker/buildAndPush.sh
> 2. ./incubator-openwhisk-runtime-go-1.13.0-incubating/examples/Makefile
> 3. ./incubator-openwhisk-runtime-python-1.13.0-incubating/core/python3AiAction/samples/smart-body-crop/fashion-men-1.jpg
>
>
> ---------------------------------------------------------------------
> 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
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
> We tried very hard to only use min-headers in cases where they were config/property/test/simple script type files (often used by tooling).
Which according to [1] is not an appropriate use, you also have source files with the short header. I suggest you check with legal if this is OK.
Thanks,
Justin
1. https://www.apache.org/legal/src-headers.html#is-a-short-form-of-the-source-header-available
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by Matt Rutkowski <mr...@apache.org>.
Hi Justin,
Given the documented rationale of where we use the mini-header (https://github.com/apache/incubator-openwhisk-release/blob/master/docs/license_compliance.md#project-asf-License-header-policy) are there specific files (types) or specific files themselves where you believe they are not appropriate uses of the header? We tried very hard to only use min-headers in cases where they were config/property/test/simple script type files (often used by tooling).
thx,
-matt
On 2019/06/09 13:50:12, Justin Mclean <ju...@classsoftware.com> wrote:
> Hi,
>
> Sorry to be clearer re the short header issue I suggest you read [1]. It may be used with mages, minified JavaScript or PDFs, that’s not the case where it’s been used in your releases and this has been pointed out several times. If there are other cases where you have confirmed with legal-discuss that OK to use then please point them out. While this is only a minor issue, but its vest to comply with ASF policy. What I consider a little more serious is that issues nee reoccurring after they have been pointed out.
>
> Thanks,
> Justin
>
> 1. https://www.apache.org/legal/src-headers.html#is-a-short-form-of-the-source-header-available
> ---------------------------------------------------------------------
> 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
Re: Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by David P Grove <gr...@us.ibm.com>.
Justin Mclean <ju...@classsoftware.com> wrote on 06/09/2019 09:50:12 AM:
>
> Sorry to be clearer re the short header issue I suggest you read
> [1]. It may be used with mages, minified JavaScript or PDFs, that’s
> not the case where it’s been used in your releases and this has been
> pointed out several times. If there are other cases where you have
> confirmed with legal-discuss that OK to use then please point them
> out. While this is only a minor issue, but its vest to comply with
> ASF policy. What I consider a little more serious is that issues nee
> reoccurring after they have been pointed out.
>
Hi Justin,
We did discuss this issue on our dev list [2] and the guidance from our
mentor [3] was that since we had documented the rationale for using short
form headers in additional cases, what we were doing was acceptable. We
have started including the URL of our project policy on header file formats
[4] in all of our release vote emails to refer voters to the project policy
on allowable forms of the license headers.
--dave
[1]
https://www.apache.org/legal/src-headers.html#is-a-short-form-of-the-source-header-available
[2]
https://lists.apache.org/thread.html/bbec59cc560d446a4239627dcb463bc95fb98be5eb66d525f93e5ef1@%3Cdev.openwhisk.apache.org%3E
[3]
https://lists.apache.org/thread.html/6d25eb12cbfad05693cda729cff62b06ba45845ad53f5d1a67a2ac97@%3Cdev.openwhisk.apache.org%3E
[4]
https://github.com/apache/incubator-openwhisk-release/blob/master/docs/license_compliance.md
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
Sorry to be clearer re the short header issue I suggest you read [1]. It may be used with mages, minified JavaScript or PDFs, that’s not the case where it’s been used in your releases and this has been pointed out several times. If there are other cases where you have confirmed with legal-discuss that OK to use then please point them out. While this is only a minor issue, but its vest to comply with ASF policy. What I consider a little more serious is that issues nee reoccurring after they have been pointed out.
Thanks,
Justin
1. https://www.apache.org/legal/src-headers.html#is-a-short-form-of-the-source-header-available
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by Rodric Rabbah <ro...@gmail.com>.
> > We've also discussed the use of "short licenses" [5] and we document our
> use of the short licenses
>
> Which I see was some time ago but this keeps happening in your releases.
>
You are correct that we have an outstanding item to tighten our automated
checks inline with the project's documented policy. But I'm not sure if
you're implying we're doing something wrong. We don't believe the use of
short license is out of line with Apache policy
https://www.apache.org/legal/src-headers.html. We document which files get
short licenses or license exclusions here:
https://github.com/apache/incubator-openwhisk-release/blob/master/docs/license_compliance.md
-r
Re: [VOTE] Release Apache OpenWhisk Runtimes v1.13.0-incubating
Posted by Justin Mclean <ju...@me.com>.
Hi,
> For several of the issues you noted, I opened github defects [1-4] against the relevant repos so that we will address them before the next release of the corresponding artifacts.
Thanks for that.
> We've also discussed the use of "short licenses" [5] and we document our use of the short licenses
Which I see was some time ago but this keeps happening in your releases.
Thanks,
Justin