RFR: 8333685: Make update-copyright-year script more useful [v3]
Erik Joelsson
erikj at openjdk.org
Thu Jun 13 13:06:15 UTC 2024
On Wed, 12 Jun 2024 14:04:41 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:
>> Hi all,
>>
>> This PR addresses [8333685](https://bugs.openjdk.org/browse/JDK-8333685).
>>
>> I have added the following enhancements:
>> - Removed uses of `fgrep` and `egrep` with `grep -F` and `grep -E` respectively.
>> - Altered default behaviour to limit the processed changesets to those in the current branch and the current year.
>> - Enabled an option to modify all changesets in the year if desired (this was the previous default behaviour).
>> - Added named parameters and enabled `--help`.
>> - Removed mercurial support.
>> - Fixed a bug where copyright headers that didn't have a comma following the initial year of creation were not getting picked up. For example, [here](https://github.com/openjdk/jdk/blob/512b2b4f141f9a202984150b0427372e1a409a50/src/hotspot/cpu/zero/copy_zero.hpp#L3).
>> - Fixed a bug where copyright headers missing the copyright symbol (c) were not getting picked up. Refer to the example above as well.
>>
>> Thanks,
>> Sonia
>
> Sonia Zaldana Calles has updated the pull request incrementally with two additional commits since the last revision:
>
> - Support for copyright headers that don't add a comma after final copyright year
> - Changes based on feedback
Marked as reviewed by erikj (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/19605#pullrequestreview-2115736731
More information about the build-dev
mailing list