Hi,

There is a similar file that is MIT licensed, showing the issue with not having 
headers on files. Yes, I read LEGAL-649. That only applies if the IP province 
and license are clear. It is not in this case. I’ve not checked the other 
files, but given that the previous owner was a bit loose with IP and licensing, 
it would be IMO best to check. MIT is compatible with the ALv2, so there is not 
a license issue here, just an ASF policy one (unless other files are shown to 
be under different licenses).

Kind Regards,
Justin

> On 5 Aug 2023, at 6:11 pm, PJ Fanning <fannin...@gmail.com> wrote:
> 
> Hi Justin,
> 
> Could you provide the evidence that this file is needs a special license?
> 
> https://github.com/akka/akka-sample-cluster-kubernetes-scala/blob/master/scripts/test.sh
> 
> There is no evidence that I can see that this file was not written by
> the Lightbend staff.
> 
> Lightbend do not typically add license headers to shell scripts used
> in their tests (like this one).
> 
> It is ASF policy not to change the license headers on source code that
> we did not write.
> 
> This was discussed in https://issues.apache.org/jira/browse/LEGAL-649
> 
> Should I reopen LEGAL-649?
> 
> 
> On Sat, 5 Aug 2023 at 03:00, Justin Mclean <jus...@classsoftware.com> wrote:
>> 
>> Hi,
>> 
>> -1(binding) due to missing headers and possible 3rd party code in the 
>> release not mentioned in LICENSE
>> 
>> I checked:
>> - incubating in name
>> - signatures and hashes are correct
>> - DISCLAIMER exists
>> - LICENSE may be missing license(s) of 3rd party code
>> - NOTICE is fine
>> - Some files are missing headers [1][2][3][4][5][6][7][8][9][10][11] and at 
>> least one of these seems to be a 3rd party file but I’ve not checked them 
>> all. The file seems to MIT licensed file [13], but perhaps it was copied 
>> from elsewhere, and its license changed?
>> - No unexpected binaries
>> - Can compile from source
>> 
>> Kind Regards,
>> Justin
>> 
>> 1. ./integration-test/aws-api-ecs/scripts/deploy-infrastructure.sh
>> 2. ./integration-test/aws-api-ecs/scripts/deploy.sh
>> 3. ./integration-test/aws-api-ecs/scripts/publish.sh
>> 4. ./integration-test/kubernetes-api/test.sh
>> 5. ./integration-test/kubernetes-dns/test.sh
>> 6  ./integration-test/scripts/kubernetes-test.sh
>> 7 ./integration-test-scripts/aws-api-ec2-tag-based.sh
>> 8. 
>> ./lease-kubernetes/src/it/scala/org/apache/pekko/coordination/lease/kubernetes/KubernetesApiIntegrationTest.scala
>> 9 
>> ./lease-kubernetes/src/it/scala/org/apache/pekko/coordination/lease/kubernetes/LeaseContentionSpec.scala
>> 10. ./lease-kubernetes-int-test/minikube-test.sh
>> 11. ./scripts/cat-log.sh
>> 12. ./scripts/authors.scala
>> 13. 
>> https://github.com/akka/akka-sample-cluster-kubernetes-scala/blob/master/scripts/test.sh
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscr...@pekko.apache.org
>> For additional commands, e-mail: dev-h...@pekko.apache.org
>> 


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

Reply via email to