[PATCH 4/4] [pre-commit] Add copyright-years check

[Tom de Vries <tdevries@suse.de>]

Use the gdb/copyright.py script to do a copyright years pre-commit check.

This complains for instance when trying to commit a new file with copyright
year 2025 in 2026 [1].

The check seems quick enough:
...
$ pre-commit run --all-files -v copyright-years
copyright-years.........................................................Passed
- hook id: copyright-years
- duration: 0.84s
...

The first time though someone tries to commit something in a new year, this
check will generate a massive amount of changes. [ Note that the script still
needs running manually, since the pre-commit check uses only part of the
script. ]

We could fix this by hardcoding the year, and require manual bumping after all
the other files are updated, but I left this as is.

[1] https://sourceware.org/pipermail/gdb-patches/2026-April/226916.html
---
 .pre-commit-config.yaml |  7 +++++++
 gdb/copyright.py        | 13 +++++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 5b1af9a1837..a53046c0e90 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -112,6 +112,13 @@ repos:
       additional_dependencies: ["pyyaml"]
       always_run: true
       require_serial: true
+    - id: copyright-years
+      name: copyright-years
+      language: python
+      entry: gdb/copyright.py
+      args: [--pre-commit]
+      files: '^(gdb|gdbsupport|gdbserver)/.*$'
+      pass_filenames: true
   - repo: https://github.com/nmoroze/tclint
     rev: v0.8.0
     hooks:
diff --git a/gdb/copyright.py b/gdb/copyright.py
index 497dd1fa93c..7f5d85a5909 100755
--- a/gdb/copyright.py
+++ b/gdb/copyright.py
@@ -181,6 +181,7 @@ def get_parser() -> argparse.ArgumentParser:
     parser = argparse.ArgumentParser(
         description=__doc__, formatter_class=argparse.RawDescriptionHelpFormatter
     )
+    parser.add_argument("--pre-commit", nargs="+")
     return parser
 
 
@@ -192,15 +193,23 @@ def main(argv: list[str]) -> int | None:
         sys.exit("Error: This script requires python >= 3.13.")
 
     parser = get_parser()
-    _ = parser.parse_args(argv)
+    args = parser.parse_args(argv)
 
     if not os.path.isfile("gnulib/import/extra/update-copyright"):
         sys.exit("Error: This script must be called from the top-level directory.")
 
-    update_list = get_update_list()
+    # Get the update list from pre-commit, or create the list.
+    update_list = args.pre_commit if args.pre_commit else get_update_list()
+
     update_list = filter_excluded(update_list)
     update_files(update_list)
 
+    if args.pre_commit:
+        # Running autoreconf requires a specific version, and asking for
+        # but not checking manual updates doesn't fit well with pre-commit.
+        # Let's just use the automatic update part.
+        return
+
     run_autoreconf()
 
     # Remind the user that some files need to be updated by HAND...
-- 
2.51.0