Skip to content

Add SPDX notices for files with copyright notice in latest version#5633

Open
jafingerhut wants to merge 6 commits into
p4lang:mainfrom
jafingerhut:add-spdx-notices-for-files-with-copyright-notice-in-latest-version
Open

Add SPDX notices for files with copyright notice in latest version#5633
jafingerhut wants to merge 6 commits into
p4lang:mainfrom
jafingerhut:add-spdx-notices-for-files-with-copyright-notice-in-latest-version

Conversation

@jafingerhut

@jafingerhut jafingerhut commented May 20, 2026

Copy link
Copy Markdown
Contributor

These are all of the remaining files in the repo that have an explicit copyright notice already. The changes preserve the copyright holder and year, and add an SPDX-License-Identifier line with license Apache-2.0. previously all of these files already had a different non-SPDX form of Apache-2.0 license, or no mention of license at all.

…tices

My code that looked for files with explicit copyright notices and were
updated in an earlier commit, did not check for whether the file had a
copyright notice in the current (i.e. latest) version of the file.
This commit adds SPDX copyright and license annotations for those
files.

Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
@fruffy fruffy added the documentation Topics related to compiler documentation. label May 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Doxygen Preview Ready!
Click here to view the preview

…est-version' of github.com:jafingerhut/p4c into add-spdx-notices-for-files-with-copyright-notice-in-latest-version
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Doxygen Preview Ready!
Click here to view the preview

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

Labels

documentation Topics related to compiler documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants