Skip to content

Fix copyright update heuristic#10860

Open
garazdawi wants to merge 3 commits intoerlang:maint-28from
garazdawi:lukas/otp/fix-copyright-update-heuristic
Open

Fix copyright update heuristic#10860
garazdawi wants to merge 3 commits intoerlang:maint-28from
garazdawi:lukas/otp/fix-copyright-update-heuristic

Conversation

@garazdawi
Copy link
Contributor

This PR fixes three issues with the copyright update script:

  1. Do not update copyrights in vendored files (even if the vendored files follow our copyright header).
  2. Correctly update the copyright info in .license files.
  3. Improve rename tracking so that copyright information can be gotten from renamed files.

Before this fix, the commit log of the .license file was used
instead of the actual file.
When updating copyright notices we want to keep the dates
of the copyright notices even if the file is renamed.
Unfortunately --follow seems to not suite our purposes
so this commit contains a version of git log --follow
that does not track copies and only allows renames within
the same commit.
@github-actions
Copy link
Contributor

github-actions bot commented Mar 16, 2026

CT Test Results

  1 files   11 suites   3m 9s ⏱️
 95 tests  91 ✅ 4 💤 0 ❌
111 runs  107 ✅ 4 💤 0 ❌

Results for commit 97161c2.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant