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.bur...@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
