Skip to content

Trim license boilerplate to the SPDX identifier - #1950

Open
davidharrishmc wants to merge 3 commits into
openhwfoundation:mainfrom
davidharrishmc:dh/spdx-license
Open

davidharrishmc wants to merge 3 commits into
openhwfoundation:mainfrom
davidharrishmc:dh/spdx-license

Conversation

@davidharrishmc

@davidharrishmc davidharrishmc commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

The full Solderpad/Apache license text lives in LICENSE. This removes the repeated license paragraph from 372 file headers and keeps the copyright line and SPDX-License-Identifier: Apache-2.0 WITH SHL-2.1. Comment-only change; no code is modified.

🤖 Generated with Claude Code

The LICENSE file holds the full Solderpad/Apache text. File headers now
keep only the copyright line and the SPDX-License-Identifier line.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@davidharrishmc

Copy link
Copy Markdown
Contributor Author

Review notes: the change is comment-only (3868 deletions, no insertions, every file keeps its SPDX line), but:

🤖 Generated with Claude Code

davidharrishmc and others added 2 commits October 16, 2026 03:32
Twenty coverage tests added on main after this branch was made still carried the
Solderpad/Apache notice paragraph; trim them to the copyright and SPDX lines like
the rest.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@davidharrishmc

Copy link
Copy Markdown
Contributor Author

Merge order: #1954 contains this PR's commit. Merge this first and then #1954, or close this one in favour of #1954. Either way, both should land after the bug-fix PRs and before the #1952 → #1955 → #1959 stack, or be resolved against it: the stack deletes four files this PR edits (arrs.sv, aesmixcolumns8.sv, rotate.sv, trickbox_apb.sv).

🤖 Generated with Claude Code

@davidharrishmc

Copy link
Copy Markdown
Contributor Author

Updated per review: merged main and trimmed the license paragraph from the 20 coverage tests added on main since this branch was made (the 9 noted in the review plus 11 that landed after it, including the wfi tests from #1941 and fpADUpdate from #1931). No file outside LICENSE still carries the paragraph. The change is still comment-only: 372 files, deletions only.

🤖 Generated with Claude Code

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