search

Tencent / TencentKona-17

Tencent Kona JDK17 is a no-cost, production-ready distribution of the Open Java Development Kit (OpenJDK), Long-Term Support(LTS) with quarterly updates. Tencent Kona JDK17 is certified as compatible with the Java SE standard.
Other
142 stars 27 forks source link

Fix copyright notes #29

Open iseri-nina opened 3 weeks ago

iseri-nina commented 3 weeks ago

This PR will resolve issue #19: Fix copyright notes.

heliang666s commented 3 weeks ago

20 老哥你把我issue的内容做了。#19 难道不是增加缺失的逗号？

iseri-nina commented 3 weeks ago

20 老哥你把我issue的内容做了。#19 难道不是增加缺失的逗号？

我看 #19 的要求里“年份之后缺少逗号”只是个举例，也没具体指定是什么问题
翻遍了也没找到漏逗号的，这里改的那部分似乎也能算是 copyright notes 里的，于是就提交了 PR

johnshajiang commented 2 weeks ago

另外，请问你是否已经签属了[Tencent Contributor License Agreement]？

[Tencent Contributor License Agreement]: https://cla.opensource.tencent.com/

johnshajiang commented 1 week ago

@iseri-nina 非常感谢你的修改。 我已经快速浏览了你的文件，后续还会仔细看看。

另外，如果PR没有问题，即便这个patch没有合并到Kona JDK中，依然可以认为你完成了这个的issue。

johnshajiang commented 1 week ago

@iseri-nina 当一个文件被修改之后，它的版本声明中的年份也需要相应的更新。 比如，原有的Copyright (C) 2021 THL A29 Limited需要修改为Copyright (C) 2021, 2024, THL A29 Limited。

iseri-nina commented 1 week ago

@johnshajiang

Copyright (C) 2020, 2022, THL A29 Limited, a Tencent company. All rights reserved. 对于这一类已经存在多个年份的版本声明，是修改成 Copyright (C) 2020, 2024, THL A29 Limited, a Tencent company. All rights reserved. 还是 Copyright (C) 2020, 2022, 2024, THL A29 Limited, a Tencent company. All rights reserved.

谢谢

johnshajiang commented 1 week ago

年份最多就两个，即起始年份和结束年份，所以是写成如下形式， Copyright (C) 2020, 2024, THL A29 Limited

iseri-nina commented 1 week ago

好的，已完成修改

johnshajiang commented 1 week ago

@iseri-nina 非常感谢你的更新。 这个补丁对于我来说，应该没有问题。

不过，考虑到有很多文件也存在于OpenJDK中，所以我们需要在未来逐步地处理它们。 但我们依然认为你完成了该issue，已经在研学基地中更新它的状态，请查看。