Copyright update tedium

[Archie Cobbs]

Thanks for working on this... something of a thankless task :)

I'm sure you've considered this but I'll ask anyway. Would it make (more or
less) sense to try and enforce the policy on the front-end?

By that I mean adding another checkbox requirement to skara's handling of
PR's: "🔲 Change must update copyright dates where applicable"

The check could start out being conservative:

   - Only applies to files with certain extensions and/or matching some
   filter list
   - Only applies to files containing a recognizable copyright text line

-Archie

On Mon, Dec 9, 2024 at 7:06 AM Magnus Ihse Bursie <
magnus.ihse.bursie at oracle.com> wrote:

> I felt responsibility for the .github files, and wanted to check if there
> were more build system files needed updating. So I ran a more comprehensive
> script, and discovered a *lot* more files that needed updating. Like a
> thousand or so...
>
> I have opened a series of issues starting at
> https://bugs.openjdk.org/browse/JDK-8345793 and going up to
> https://bugs.openjdk.org/browse/JDK-8345805 to update these headers.
>
> I agree, this should be automated. We're starting to slowly get there, see
> https://github.com/openjdk/skara/pull/1702 for a first step.
>
> /Magnus
> On 2024-12-03 16:45, Archie Cobbs wrote:
>
> Dumb question...
>
> It seems like the thing with updating copyright years in source files
> could be better automated. At least, couldn't there be a test that fails if
> you forget?
>
> FWIW my little updater script says that these files still need to be
> updated to 2024:
>
> .github/actions/config/action.yml
> .github/actions/do-build/action.yml
> .github/actions/get-bootjdk/action.yml
> .github/actions/get-bundles/action.yml
> .github/actions/get-msys2/action.yml
> .github/scripts/gen-build-failure-report.sh
> .github/scripts/gen-test-summary.sh
> .github/workflows/build-cross-compile.yml
> .github/workflows/test.yml
> src/java.base/share/classes/jdk/internal/org/xml/sax/Attributes.java
> src/java.base/share/classes/jdk/internal/org/xml/sax/InputSource.java
> src/java.base/share/classes/jdk/internal/org/xml/sax/SAXException.java
> src/java.base/share/classes/jdk/internal/org/xml/sax/SAXParseException.java
> src/java.base/share/classes/jdk/internal/org/xml/sax/XMLReader.java
>
> src/java.base/share/classes/jdk/internal/org/xml/sax/helpers/DefaultHandler.java
> src/java.base/share/classes/jdk/internal/platform/Metrics.java
> test/hotspot/jtreg/compiler/vectorization/runner/BasicIntOpTest.java
> test/hotspot/jtreg/compiler/vectorization/runner/BasicLongOpTest.java
> test/hotspot/jtreg/compiler/whitebox/DeoptimizeFramesTest.java
>
> -Archie
>
> --
> Archie L. Cobbs
>
>

-- 
Archie L. Cobbs
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/core-libs-dev/attachments/20241209/6151a4e7/attachment-0001.htm>