Re: RFR: 8333685: Make update-copyright-year script more useful [v2]

2024-06-12 Thread Sonia Zaldana Calles
On Wed, 12 Jun 2024 06:46:39 GMT, Thomas Stuefe wrote: > A lot better, but not fully there yet. Can you make it so that it also picks > up SAP's copyrights when they specify multiple years? They are weird in that > the last year does not have a trailing comma, e.g.: > > ``` > * Copyright (c)

Re: RFR: 8333685: Make update-copyright-year script more useful [v2]

2024-06-12 Thread Sonia Zaldana Calles
On Wed, 12 Jun 2024 06:43:15 GMT, Thomas Stuefe wrote: >> Sonia Zaldana Calles has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fixing optional group to work with bsd as well > > make/scripts/update_copyright_year.sh line 70: > >> 68:

Re: RFR: 8333685: Make update-copyright-year script more useful [v2]

2024-06-12 Thread Thomas Stuefe
On Tue, 11 Jun 2024 13:34:24 GMT, Sonia Zaldana Calles 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.

Re: RFR: 8333685: Make update-copyright-year script more useful [v2]

2024-06-11 Thread Erik Joelsson
On Tue, 11 Jun 2024 13:34:24 GMT, Sonia Zaldana Calles 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.

Re: RFR: 8333685: Make update-copyright-year script more useful [v2]

2024-06-11 Thread Sonia Zaldana Calles
> 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